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

Allow for more complex version comparison #577

Closed

Conversation

mturzanska
Copy link
Contributor

Allow for letters in version names
Add tests for version info

Addresses #557

Allow for letters in version names
Add tests for version info
@drewbanin
Copy link
Contributor

Let's try to combine this PR with the version comparison code over here: #542

cc @cmcarthur

@drewbanin
Copy link
Contributor

hey @mturzanska - we've been bad about getting these PRs merged, but I'm going to have some time to dig in this week. Do you mind if I try to take this over the line?

@mturzanska
Copy link
Contributor Author

@drewbanin sure, go ahead

@drewbanin
Copy link
Contributor

Closing this in favor of #797

Thanks for your help here @mturzanska -- I'll be sure to give you credit for this one in the release notes :)

@drewbanin drewbanin closed this Jun 18, 2018
drewbanin added a commit that referenced this pull request Jun 27, 2018
* Allow for more complex version comparison

Allow for letters in version names
Add tests for version info

* Fix integration tests

* Combine code from @mturzanska's PR with development

Based on: #577
Fixes: #557
iknox-fa pushed a commit that referenced this pull request Feb 8, 2022
* Allow for more complex version comparison

Allow for letters in version names
Add tests for version info

* Fix integration tests

* Combine code from @mturzanska's PR with development

Based on: #577
Fixes: #557


automatic commit by git-black, original commits:
  d28407d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants