Skip to content

Wr/activate version @W-21429701@#348

Merged
marcelinollano merged 14 commits intomainfrom
wr/activateVersion
Mar 11, 2026
Merged

Wr/activate version @W-21429701@#348
marcelinollano merged 14 commits intomainfrom
wr/activateVersion

Conversation

@WillieRuemmele
Copy link
Contributor

What does this PR do?

let's users select which version to activate, adds --json adds errors/warnings when using --json without all flags

What issues does this PR fix or reference?

@W-21429701@
requires: forcedotcom/agents#228

@WillieRuemmele WillieRuemmele requested a review from a team as a code owner March 3, 2026 17:49
Copy link
Collaborator

@marcelinollano marcelinollano left a comment

Choose a reason for hiding this comment

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

Tested locally and everything works well. A couple of minor things I noticed for consistency:

  1. The activate command has the new --version flag, but deactivate doesn't. Currently deactivate auto-detects the active version which works fine, but you might want to add it for symmetry.

  2. activate has enableJsonFlag = true but deactivate doesn't (though --json still works on both).

@marcelinollano marcelinollano merged commit 6327eb5 into main Mar 11, 2026
15 checks passed
@marcelinollano marcelinollano deleted the wr/activateVersion branch March 11, 2026 17:33
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