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
changed easyconfig.update to be able to specify not to allow duplicate values #2657
changed easyconfig.update to be able to specify not to allow duplicate values #2657
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Makes sense to add support for this, but it does require a test (and a fix, see other comment) before it can go in...
Tests included. Fixed the list problem. |
Mmmm, @boegel, am I noticing boegelbot acting stupid ? I don't understand the error log. |
@mboisson No, it's correctly reporting a failing test. Try running the failing test locally with |
Ok, seems the tests all passed. Ready for another review @boegel |
lgtm, thanks @mboisson! |
This is to address easybuilders/easybuild-easyblocks#1574