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

[fix] Update ynh_handle_getopts_args #628

Merged
merged 2 commits into from Jan 28, 2019

Conversation

Projects
None yet
2 participants
@maniackcrudelis
Copy link
Contributor

maniackcrudelis commented Jan 27, 2019

The problem

ynh_handle_getopts_args has some bugs

Solution

Fix ynh_handle_getopts_args from #561

PR Status

Ready to be reviewed

Validation

  • Principle agreement 0/2 :
  • Quick review 0/1 :
  • Simple test 0/1 :
  • Deep review 0/1 :
Show resolved Hide resolved data/helpers.d/getopts Outdated
Show resolved Hide resolved data/helpers.d/getopts Outdated
@alexAubin
Copy link
Member

alexAubin left a comment

LGTM

@alexAubin alexAubin added this to the 3.4.x milestone Jan 28, 2019

@alexAubin alexAubin merged commit 375a268 into stretch-unstable Jan 28, 2019

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@alexAubin alexAubin deleted the fix_getops_helper branch Jan 28, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment