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

Added update command #960

Merged
merged 11 commits into from
Nov 15, 2024
Merged

Added update command #960

merged 11 commits into from
Nov 15, 2024

Conversation

petarjuki7
Copy link
Member

App update command

@MatejVukosav
Copy link
Member

MatejVukosav commented Nov 10, 2024

Gj! Can you also add to the docs how to use it?

Copy link
Member

@miraclx miraclx left a comment

Choose a reason for hiding this comment

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

so.. we need the following:

$ meroctl context update --context-id <context-id> -a <application-id> --as <identity>
# one-off update

$ meroctl context update --context-id <context-id> --path ./path/to/file --as <identity>
# one-off update

$ meroctl context update --context-id <context-id> --path ./path/to/file --watch --as <identity>
# recurring update

see the call command for how identity is managed

<application-id> and <path> are mutually exclusive, and --watch should only be allowed when --path is present.

crates/meroctl/src/cli/context/update.rs Outdated Show resolved Hide resolved
crates/meroctl/src/cli/context/update.rs Outdated Show resolved Hide resolved
crates/meroctl/src/cli/context/update.rs Outdated Show resolved Hide resolved
@petarjuki7 petarjuki7 requested a review from miraclx November 14, 2024 20:32
@petarjuki7 petarjuki7 enabled auto-merge (squash) November 15, 2024 07:43
@petarjuki7 petarjuki7 merged commit a6f6bfa into master Nov 15, 2024
15 checks passed
@petarjuki7 petarjuki7 deleted the petarjuki7/cli-context-update branch November 15, 2024 08:14
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.

3 participants