-
Notifications
You must be signed in to change notification settings - Fork 94
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
download: add --srpm
option
#1207
download: add --srpm
option
#1207
Conversation
4f5c868
to
a724d6e
Compare
Thanks for the feedback! I updated the PR. I'm not sure if this is the best way to do this, but I couldn't find something else. Edit: Also renamed variables and commit message |
a724d6e
to
a7624cb
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One last thing.
(I have also prepared a PR with enabled CI tests once we merge this: rpm-software-management/ci-dnf-stack#1443) |
Could you also add a note to
|
a7624cb
to
311e2e6
Compare
311e2e6
to
146145a
Compare
Sorry for the delay, I was on vacation. I've made the requested changes and rebased the branch, thank you for the review! |
Thank you for the contribution! |
78d7b76
Fixes #947
Implements the
--source
option for the download command to download only the source rpm(s).