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.

Sign up for GitHub

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

Jump to bottom

support numeric and alphanumeric qualifiers #3

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

support numeric and alphanumeric qualifiers #3

wants to merge 4 commits into from

Conversation

bagheera
Copy link

  1. Small fix to parse numeric qualifiers (e.g. 1.2.3-4). Please see testQualifierDashNumeric
  2. Number-aware comparison of qualifiers with numbers (e.g. 1.2.3-r10 is greater than 1.2.3-r9). Made use of Stephen Friedrich's BSD licensed Strings utility for this. Please see testCompareQualifier and testCompareQualifierNumeric

thanks
sriram

@buildhive
Copy link

Michael Rumpf » repoclient-plugin #6 SUCCESS
This pull request looks good
(what's this?)

@mrumpf
Copy link
Owner

mrumpf commented Aug 12, 2013

The Strings class should be moved into the plugins' package hierarchy in order to avoid potential clashes with other plugins.

@buildhive
Copy link

Michael Rumpf » repoclient-plugin #10 UNSTABLE
Looks like there's a problem with this pull request
(what's this?)

@buildhive
Copy link

Michael Rumpf » repoclient-plugin #11 SUCCESS
This pull request looks good
(what's this?)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@bagheera @buildhive @mrumpf @go01