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

Global install_resolver_dependencies: false in yaml file does not work #180

Open
mvdbeek opened this issue Sep 22, 2021 · 0 comments · May be fixed by #181
Open

Global install_resolver_dependencies: false in yaml file does not work #180

mvdbeek opened this issue Sep 22, 2021 · 0 comments · May be fixed by #181
Assignees
@mvdbeek
Labels
bug

Comments

@mvdbeek
Copy link
Member

mvdbeek commented Sep 22, 2021

ephemeris/src/ephemeris/shed_tools_args.py

Line 123 in c6aea18

default=True, # Override False default for this function
with https://github.com/galaxyproject/ephemeris/blob/master/src/ephemeris/shed_tools.py#L609 means the command line argument default value takes precedence, which is True.
@mvdbeek mvdbeek added the bug label Sep 22, 2021
@mvdbeek mvdbeek self-assigned this Sep 22, 2021
mvdbeek added a commit to mvdbeek/ephemeris that referenced this issue Sep 23, 2021
@mvdbeek
Fix precedence of commandline options and yaml options
6f968e2 
commandline options should take precedence over options specied in yaml
file, but default values should be used if options are not set in yaml
file. Fixes galaxyproject#180
@mvdbeek mvdbeek linked a pull request Sep 23, 2021 that will close this issue
Open
mvdbeek added a commit to mvdbeek/ephemeris that referenced this issue Sep 23, 2021
@mvdbeek
Fix precedence of commandline options and yaml options
62ff309 
commandline options should take precedence over options specied in yaml
file, but default values should be used if options are not set in yaml
file. Fixes galaxyproject#180
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mvdbeek mvdbeek

Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix precedence of commandline options and yaml options mvdbeek/ephemeris
1 participant
@mvdbeek