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

Pipx support #3

Merged
merged 1 commit into from
Oct 21, 2021
Merged

Pipx support #3

merged 1 commit into from
Oct 21, 2021

Conversation

henryiii
Copy link
Contributor

@henryiii henryiii commented Oct 20, 2021

For any package that has a different command line interface, pipx run is much more pleasant if you also add an entry point for it. (not needed if the names match). See pypa/pipx#600

PS: I'd also recommend moving to setup.cfg, easier to work with and maintain. See https://packaging.python.org or https://scikit-hep.org/developer for examples.

For any package that has a different command line interface, pipx run is much more pleasant if you also add an entry point for it. (not needed if the names match).
@restyled-io restyled-io bot mentioned this pull request Oct 20, 2021
@wasi-master
Copy link
Owner

Thanks

@wasi-master wasi-master merged commit 6d28a9d into wasi-master:main Oct 21, 2021
@wasi-master
Copy link
Owner

wasi-master commented Oct 21, 2021

And yeah I've been thinking about switching to setup.cfg for a while, I'll do it in a moment

@wasi-master
Copy link
Owner

Yeah I switched to setup.cfg

@henryiii
Copy link
Contributor Author

FYI, just in case you find it useful, using setup.cfg, setup-cfg-fmt is really handy to keep it formatted and up to date with classifiers and such. See https://github.com/asottile/setup-cfg-fmt or https://scikit-hep.org/developer/style#setupcfg-format-extra.

@henryiii henryiii deleted the patch-1 branch October 21, 2021 17:08
@wasi-master
Copy link
Owner

Thanks for the suggestion, I'll look into it.

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.

None yet

2 participants