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(reactive): run charm build in verbose mode #2021

Merged
merged 1 commit into from
Dec 11, 2024
Merged

Conversation

lengau
Copy link
Collaborator

@lengau lengau commented Dec 10, 2024

While a user could previously have added --verbose to reactive-charm-build-arguments, this automatically adds --verbose if the user hasn't specified a log level.

The verbose text gets handled by craft-cli anyway, but this changes the default so the text gets added to the log.

Spread test failures are unrelated: #2023

Fixes #924

@lengau lengau force-pushed the work/924/CRAFT-3424 branch from 9977710 to e41c932 Compare December 10, 2024 15:22
While a user could previously have added `--verbose` to
`reactive-charm-build-arguments`, this automatically adds `--verbose`
if the user hasn't specified a log level.

The verbose text gets handled by craft-cli anyway, but this changes the
default so the text gets added to the log.
@lengau lengau force-pushed the work/924/CRAFT-3424 branch from e41c932 to 35414e4 Compare December 10, 2024 15:29
@lengau lengau marked this pull request as ready for review December 10, 2024 15:41
@lengau lengau requested review from bepri and mr-cal December 10, 2024 15:41
Copy link
Contributor

@bepri bepri left a comment

Choose a reason for hiding this comment

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

Looks good!

Copy link
Contributor

@mr-cal mr-cal left a comment

Choose a reason for hiding this comment

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

Very clean!

@lengau lengau merged commit 7129d57 into main Dec 11, 2024
22 of 23 checks passed
@lengau lengau deleted the work/924/CRAFT-3424 branch December 11, 2024 18:05
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.

reactive plugin: call charm build with -v
3 participants