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

Jump to bottom

kamel rebuild default should not rebuild all Integrations #3445

Closed
squakez opened this issue Jul 13, 2022 · 1 comment · Fixed by #3556
Closed

kamel rebuild default should not rebuild all Integrations #3445

squakez opened this issue Jul 13, 2022 · 1 comment · Fixed by #3556
Assignees
@essobedo
Labels
area/cli Kamel CLI area/ux Improves user experience good first issue Does not require full understanding of the codebase
Milestone
1.10.0

Comments

@squakez
Copy link
Contributor

squakez commented Jul 13, 2022

I think it is a bit dangerous to leave kamel rebuild default behavior to take in account all Integrations. We should instead have a default that will ask for more parameter (ie, the Integration name or a --all flag which updates all Integrations).
@squakez squakez added good first issue Does not require full understanding of the codebase area/ux Improves user experience area/cli Kamel CLI labels Jul 13, 2022
@essobedo essobedo self-assigned this Aug 19, 2022
@essobedo essobedo mentioned this issue Aug 19, 2022
Merged
@essobedo
Copy link
Contributor

The corresponding PR #3556

@tadayosi tadayosi added this to the 1.10.0 milestone Aug 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@essobedo essobedo

Labels
area/cli Kamel CLI area/ux Improves user experience good first issue Does not require full understanding of the codebase
Projects
None yet
Milestone
1.10.0
Development

Successfully merging a pull request may close this issue.

feat(cli): Force arguments for the rebuild command apache/camel-k
3 participants
@tadayosi @essobedo @squakez