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

New package: python3-typer-0.9.0 #48472

Closed
wants to merge 1 commit into from
Closed

New package: python3-typer-0.9.0 #48472

wants to merge 1 commit into from

Conversation

c0m4r
Copy link
Contributor

@c0m4r c0m4r commented Feb 1, 2024

Typer, build great CLIs. Easy to code. Based on Python type hints.
https://github.com/tiangolo/typer

Testing the changes

  • I tested the changes in this PR: YES

New package

Local build testing

  • I built this PR locally for these architectures (if supported. mark crossbuilds):
    • x86_64-glibc
    • x86_64-musl

@classabbyamp classabbyamp added the new-package This PR adds a new package label Feb 1, 2024
@classabbyamp
Copy link
Member

are there applications that you want to package that use the python libraries you're creating new package PRs for? if not (and especially if these are for projects you are developing), you're better off installing these libraries in a python virtual environment or with pipx. adding python libraries that no packages use adds to maintainer burden with little benefit.

@c0m4r
Copy link
Contributor Author

c0m4r commented Feb 1, 2024

Yes, ultimately I plan to package a vulnerability scanner: safety, which depends on these.

@classabbyamp
Copy link
Member

then you should make one pr with that package and all its dependencies

@c0m4r
Copy link
Contributor Author

c0m4r commented Feb 1, 2024

I was about to add safety-specific dependencies in a separate PR, however these are not strictly related to safety and can well be used in other projects. There are plenty of Python3 modules in the repository like these, even such that no package depends on, and only a handful of which are apps on itself.

@classabbyamp
Copy link
Member

these are not strictly related to safety and can well be used in other projects

doesn't matter, just do it all in one. if you want to package safety, you should just do it. having a bunch of PRs that depend on each other will make it harder to get merged.

@c0m4r
Copy link
Contributor Author

c0m4r commented Feb 1, 2024

image

@c0m4r c0m4r closed this Feb 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new-package This PR adds a new package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants