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

feat: add a 'remote-build' command #586

Merged
merged 7 commits into from
Dec 13, 2024
Merged

feat: add a 'remote-build' command #586

merged 7 commits into from
Dec 13, 2024

Conversation

tigarmo
Copy link
Contributor

@tigarmo tigarmo commented Dec 10, 2024

The implementation comes from Charmcraft. The first commit has the Charmcraft implementation unchanged, and the second commit has the code changes to make the command craft-tool-agnostic, plus some general test and linting fixes
and improvements.

@tigarmo tigarmo force-pushed the work/remote-build-command branch 2 times, most recently from c7d1079 to a53df9b Compare December 10, 2024 19:56
This is a 'chore' commit because the command is added verbatim, including
references to charmcraft entities. Also includes a helper function and
its tests, both also unchanged.
The implementation comes from Charmcraft. This commit has the code changes to
make the command craft-tool-agnostic, plus some general test and linting fixes
and improvements.
@tigarmo tigarmo force-pushed the work/remote-build-command branch from a53df9b to 699920d Compare December 11, 2024 15:17
@tigarmo tigarmo marked this pull request as ready for review December 11, 2024 15:26
@tigarmo tigarmo requested review from dariuszd21 and bepri December 11, 2024 17:03
craft_application/commands/remote.py Outdated Show resolved Hide resolved
craft_application/commands/remote.py Show resolved Hide resolved
tests/unit/commands/test_remote.py Show resolved Hide resolved
@tigarmo tigarmo requested a review from bepri December 12, 2024 18:08
Copy link
Contributor

@lengau lengau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM; nice tests!

craft_application/commands/remote.py Show resolved Hide resolved
craft_application/commands/remote.py Outdated Show resolved Hide resolved
Copy link
Contributor

@dariuszd21 dariuszd21 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

craft_application/commands/remote.py Outdated Show resolved Hide resolved
craft_application/commands/remote.py Outdated Show resolved Hide resolved
tests/unit/util/test_cli.py Outdated Show resolved Hide resolved
craft_application/commands/remote.py Show resolved Hide resolved
@tigarmo tigarmo added the squash label Dec 13, 2024
@tigarmo tigarmo merged commit 6e49d79 into main Dec 13, 2024
10 checks passed
@tigarmo tigarmo deleted the work/remote-build-command branch December 13, 2024 14:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants