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

fix(cli): explicitly set the version so --version works #11461

Merged
merged 1 commit into from
Sep 6, 2024

Conversation

Josh-Walker-GM
Copy link
Collaborator

For whatever reason the --version flag broke in v8. The code to explicitly set the version for yargs is 2 lines so we may as well be explicit and in doing so we fix the problem.

@Josh-Walker-GM Josh-Walker-GM added release:fix This PR is a fix changesets-ok Override the changesets check labels Sep 6, 2024
@Josh-Walker-GM Josh-Walker-GM added this to the next-release-patch milestone Sep 6, 2024
@Josh-Walker-GM Josh-Walker-GM self-assigned this Sep 6, 2024
@Josh-Walker-GM Josh-Walker-GM enabled auto-merge (squash) September 6, 2024 22:01
@Josh-Walker-GM Josh-Walker-GM merged commit b8ebfc8 into main Sep 6, 2024
49 of 56 checks passed
@Josh-Walker-GM Josh-Walker-GM deleted the jgmw/fix-cli-version-output branch September 6, 2024 22:10
Josh-Walker-GM added a commit that referenced this pull request Sep 6, 2024
For whatever reason auto merge just rammed #11461 right in even though
the server tests failed.
Josh-Walker-GM added a commit that referenced this pull request Sep 6, 2024
For whatever reason the `--version` flag broke in v8. The code to
explicitly set the version for yargs is 2 lines so we may as well be
explicit and in doing so we fix the problem.
Josh-Walker-GM added a commit that referenced this pull request Sep 6, 2024
For whatever reason auto merge just rammed #11461 right in even though
the server tests failed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changesets-ok Override the changesets check release:fix This PR is a fix
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

1 participant