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 logout, login to help message #699

Merged
merged 2 commits into from
Mar 30, 2022
Merged

Conversation

JacobMGEvans
Copy link
Contributor

Currently groups() on yargs does not work with Commands. However, bringing logout and login into the help message will still keep continuity between documentation and the CLI.

@JacobMGEvans JacobMGEvans added the polish Small improvements to the experience label Mar 25, 2022
@changeset-bot
Copy link

changeset-bot bot commented Mar 25, 2022

🦋 Changeset detected

Latest commit: a008418

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
wrangler Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions
Copy link
Contributor

github-actions bot commented Mar 25, 2022

A wrangler prerelease is available for testing. You can install this latest build in your project with:

npm install --save-dev https://prerelease-registry.developers.workers.dev/runs/2060844034/npm-package-wrangler-699

You can reference the automatically updated head of this PR with:

npm install --save-dev https://prerelease-registry.developers.workers.dev/prs/699/npm-package-wrangler-699

Or you can use npx with this latest build directly:

npx https://prerelease-registry.developers.workers.dev/runs/2060844034/npm-package-wrangler-699 dev path/to/script.js

@JacobMGEvans JacobMGEvans force-pushed the helpstring-commands branch 2 times, most recently from 0712f3a to bb8c073 Compare March 25, 2022 20:30
@threepointone
Copy link
Contributor

imo, these should be at the end of the list, they're not commands that'l be called often.

@petebacondarwin
Copy link
Contributor

I agree that we should move these three commands lower down. Needs a rebase.

@JacobMGEvans JacobMGEvans linked an issue Mar 29, 2022 that may be closed by this pull request
Copy link
Contributor

@petebacondarwin petebacondarwin left a comment

Choose a reason for hiding this comment

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

LGTM - thanks for fixing this @JacobMGEvans

@petebacondarwin petebacondarwin merged commit ea8e701 into main Mar 30, 2022
@petebacondarwin petebacondarwin deleted the helpstring-commands branch March 30, 2022 12:08
@github-actions github-actions bot mentioned this pull request Mar 30, 2022
mrbbot pushed a commit that referenced this pull request Oct 31, 2023
* Ensure path is relative

* Add regression test
mrbbot pushed a commit that referenced this pull request Nov 1, 2023
* Ensure path is relative

* Add regression test
mrbbot pushed a commit that referenced this pull request Nov 1, 2023
* Ensure path is relative

* Add regression test
mrbbot pushed a commit that referenced this pull request Nov 1, 2023
* Ensure path is relative

* Add regression test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
polish Small improvements to the experience
Projects
None yet
Development

Successfully merging this pull request may close these issues.

💬 RFC: Login/Logout hidden commands
3 participants