-
-
Notifications
You must be signed in to change notification settings - Fork 812
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Order of projects in Git and GitHub modules not consistant #551
Comments
@yelvert Are you using the map version of the config, or the list version for github? Maps in go do not guarantee order. I haven't seen this behavior using the list version. For git, it looks like we are deliberately sorting alphabetically for some reason. I've opened a PR to review that behavior |
For git, I’m using a list, for github, a map.
… On Sep 5, 2019, at 7:03 AM, Sean Smith ***@***.***> wrote:
@yelvert Are you using the map version of the config, or the list version for github? Maps in go do not guarantee order. I haven't seen this behavior using the list version.
For git, it looks like we are deliberately sorting alphabetically for some reason. I've opened a PR to review that behavior
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
|
I'd switch to list for github. With a map there is no way to guarantee the order in the config will be the same as the order we end up with. |
I don’t remember off hand, but there was a reason I had to use a map. I’ll try a list though, and see if that works.
… On Sep 5, 2019, at 11:37 AM, Sean Smith ***@***.***> wrote:
I'd switch to list for github. With a map there is no way to guarantee the order in the config will be the same as the order we end up with.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
|
Sounds good. If you have problems lemme know and we'll figure that part out too. |
I've fixed the GitHub documentation to show the list ordering https://wtfutil.com/modules/github/ Between using a list for |
What's broken?
The order of the repos/projects in both the Git and GitHub modules changes every time wtfutil is restarted or the config file is reloaded.
For Git, I would expect the order of the repos to match the order of the array in the config file.
For GitHub, maybe the list of repos in the config file should be changed to an array, then display them in the module in that order.
The text was updated successfully, but these errors were encountered: