Sorting by active monthly desc. #6

Merged
nutomic merged 1 commits from sort_active_monthly into main 1 year ago
Owner
There is no content yet.
dessalines added 1 commit 1 year ago
Owner

I had sorting implemented, but dont think that it should be handled by this tool, because there are a lot of different possibilities for sorting. I think it makes more sense to handle that in the program which uses the data.

I had sorting implemented, but dont think that it should be handled by this tool, because there are a lot of different possibilities for sorting. I think it makes more sense to handle that in the program which uses the data.
Poster
Owner

I def think sorting should be part of this tool, because otherwise the output has an unpredictable order, and the main point of this is to create an output file that can be placed into a git repo, with predictable changes. Having a predictable sort is a sane default.

I def think sorting should be part of this tool, because otherwise the output has an unpredictable order, and the main point of this is to create an output file that can be placed into a git repo, with predictable changes. Having a predictable sort is a sane default.
Owner

I'm not sure if thats the main point of this tool. And in that case, it seems more reliable to sort by instance name, no?

I'm not sure if thats the main point of this tool. And in that case, it seems more reliable to sort by instance name, no?
Poster
Owner

Ideally you could pass in a sort as a command line param. Names could work, but I think in the case of a crawler, where you're potentially bound to get data that could be unused or irrelevant, popularity sorts make the best sense.

Ideally you could pass in a sort as a command line param. Names could work, but I think in the case of a crawler, where you're potentially bound to get data that could be unused or irrelevant, popularity sorts make the best sense.
nutomic merged commit fefd708777 into main 1 year ago
The pull request has been merged as fefd708777.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.