Skip to content

Bug: rc1 is greater than rc11#51

Closed
nodefortytwo wants to merge 1 commit into
hashicorp:masterfrom
nodefortytwo:patch-2
Closed

Bug: rc1 is greater than rc11#51
nodefortytwo wants to merge 1 commit into
hashicorp:masterfrom
nodefortytwo:patch-2

Conversation

@nodefortytwo
Copy link
Copy Markdown

I believe that 1.0.0-rc1 is a fairly standard semver format and one would expect that rc11 is greater than rc1

@hashicorp-cla
Copy link
Copy Markdown

hashicorp-cla commented Mar 20, 2019

CLA assistant check
All committers have signed the CLA.

@radeksimko radeksimko added the bug label Apr 22, 2019
@radeksimko radeksimko changed the title add tests Bug: rc1 is greater than rc11 Apr 22, 2019
@ericmillin
Copy link
Copy Markdown

@cotarg Can you or someone else merge this? It's a breaking bug in a very common situation.

@jefferai
Copy link
Copy Markdown
Member

@ericmillin This is only an update to test files. This isn't a bug in the library itself.

@jefferai
Copy link
Copy Markdown
Member

Or to be more clear -- if it is a bug in the library itself, this PR doesn't address it.

@nodefortytwo
Copy link
Copy Markdown
Author

As you can see the original title was simply to add tests to ensure this case is considered. there is no bug in the library.

@nodefortytwo nodefortytwo marked this pull request as ready for review June 12, 2020 07:20
@nodefortytwo
Copy link
Copy Markdown
Author

Seems crazy that adding two test cases should take years to merge :)

@nodefortytwo
Copy link
Copy Markdown
Author

closing this as its annoying to have in my open PR list for years..

@nodefortytwo nodefortytwo deleted the patch-2 branch May 21, 2021 22:12
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.

5 participants