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.

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

Prefer dashes instead of underscores in flags #191

Merged
merged 1 commit into from
Apr 18, 2023

Conversation

natefoo
Copy link
Member

@natefoo natefoo commented Dec 14, 2022

This converts all ephemeris command flags that use underscores to dashes instead. Underscores continue to be supported but are hidden from the help text.

Underscores in UNIX command line flags are pretty non-standard, and are way more painful to type, to boot.

Also, entry points for install-tool-deps and set-library-permissions are installed in addition to their original underscore versions. The underscore versions of these two commands should be considered deprecated maybe be removed at some point, but the underscore flags can be kept indefinitely with little harm.

@mvdbeek
Copy link
Member

mvdbeek commented Apr 6, 2023

It's pretty conflicted now, but happy to merge if you can resolve these

@natefoo natefoo force-pushed the underscore-flags-to-dashes branch 2 times, most recently from c559ecd to 0f7b742 Compare April 6, 2023 20:45
@natefoo natefoo force-pushed the underscore-flags-to-dashes branch from 0f7b742 to f75834f Compare April 6, 2023 20:49
@natefoo
Copy link
Member Author

natefoo commented Apr 6, 2023

Done

@mvdbeek mvdbeek merged commit fa48eac into galaxyproject:master Apr 18, 2023
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