Skip to content

Fix ignoreMissingOldVersion and NewVersion check#175

Merged
siom79 merged 1 commit intosiom79:masterfrom
UnasZole:fixIgnoreMissingCheck
Sep 13, 2017
Merged

Fix ignoreMissingOldVersion and NewVersion check#175
siom79 merged 1 commit intosiom79:masterfrom
UnasZole:fixIgnoreMissingCheck

Conversation

@UnasZole
Copy link
Copy Markdown
Contributor

@UnasZole UnasZole commented Aug 4, 2017

Fix for issue #174

@UnasZole UnasZole force-pushed the fixIgnoreMissingCheck branch from e021f57 to 68a59e0 Compare August 4, 2017 08:42
@UnasZole UnasZole force-pushed the fixIgnoreMissingCheck branch from 68a59e0 to ed1c9b3 Compare August 4, 2017 08:47
@UnasZole
Copy link
Copy Markdown
Contributor Author

UnasZole commented Aug 4, 2017

I also adapted the JUnits to match my understanding of how these options should have worked.
That being said, maybe I just didn't understand how it was supposed to work, since that's absolutely not what was tested originally ^^'

Please review, and tell me what is the correct behaviour supposed to be !

Also, if the fix is accepted, how long do you think it will take to get a new release with the fix (0.10.1 I guess ?) on Maven Central ? I'd need this to work as soon as possible !
(Contrary to my PR #172 which is a "nice to have" feature that I will finish developing soon, issue #174 is actually quite blocking for me)

@siom79 siom79 merged commit e8414ea into siom79:master Sep 13, 2017
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