Skip to content
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

Sort Roles by ID instead of Version names #6

Closed
wants to merge 1 commit into from

Conversation

brian316
Copy link

I don't have ansible experience but after watching your video I thought I could help. After looking into the Galaxy API I noticed that the id is greater in the new versions so by sorting the id instead it removes any complexity and bugs in sorting by the names e.g. 1.0.9

@brian316
Copy link
Author

fixes issue GH-5

@n1ngu
Copy link

n1ngu commented Oct 13, 2021

But what if a project maintains LTS series or something like that so they could potentially release a 1.X version long after a 2.0 series was started?

@stale
Copy link

stale bot commented Jan 13, 2022

This pull request has been marked 'stale' due to lack of recent activity. If there is no further activity, the PR will be closed in another 30 days. Thank you for your contribution!

Please read this blog post to see the reasons why I mark pull requests as stale.

@stale stale bot added the stale label Jan 13, 2022
@stale
Copy link

stale bot commented Feb 17, 2022

This pull request has been closed due to inactivity. If you feel this is in error, please reopen the pull request or file a new PR with the relevant details.

@stale stale bot closed this Feb 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants