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
Support for bundles in the Upload command. #246
Support for bundles in the Upload command. #246
Conversation
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.
Looks good, just a couple of docstring suggestions
Push a charm or bundle to Charmhub where it will be verified for | ||
conformance to the packaging standard. This command will finish | ||
successfully once the package is approved by Charmhub. |
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.
Push a charm or bundle to Charmhub where it will be verified for | |
conformance to the packaging standard. This command will finish | |
successfully once the package is approved by Charmhub. | |
Push a charm or bundle to Charmhub where it will be verified. | |
This command will finish successfully once the package is | |
approved by Charmhub. |
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.
improved
charmcraft/utils.py
Outdated
"""Convert the string to Path and verify that is a useful file path. | ||
|
||
It checks that the file exists and it's readable, and that it's actually a | ||
file. Also the `~` is expanded to the user's home. | ||
""" |
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.
"""Convert the string to Path and verify that is a useful file path. | |
It checks that the file exists and it's readable, and that it's actually a | |
file. Also the `~` is expanded to the user's home. | |
""" | |
"""Return a valid Path with user name expansion for filepath. | |
CommandError is raised if filepath is not a valid file or is not readable. | |
""" |
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.
better, thanks!
Part of #128.
Fixes #77.