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

chore(cli): update help copy for deploy command #7624

Merged
merged 1 commit into from
Oct 21, 2024

Conversation

markmichon
Copy link
Contributor

Description

Updates the help text for sanity deploy to include the -y, --yes flag (unattended mode), which deploy inherits from build.

What to review

Is this indeed an intended flag for deploy and if so does the description, taken from build, adequately describe the flag.

Testing

I ran a manual test with the flag set to confirm it works as described

Notes for release

Not required

@markmichon markmichon requested a review from a team as a code owner October 16, 2024 21:16
@markmichon markmichon requested review from binoy14 and removed request for a team October 16, 2024 21:16
Copy link

vercel bot commented Oct 16, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
page-building-studio ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 16, 2024 9:20pm
performance-studio ✅ Ready (Inspect) Visit Preview Oct 16, 2024 9:20pm
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
studio-workshop ⬜️ Ignored (Inspect) Visit Preview Oct 16, 2024 9:20pm

Copy link

vercel bot commented Oct 16, 2024

@markmichon is attempting to deploy a commit to the Sanity Sandbox Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Contributor

@binoy14 binoy14 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!

@rexxars rexxars added this pull request to the merge queue Oct 21, 2024
Merged via the queue into sanity-io:next with commit a4bd01d Oct 21, 2024
38 of 54 checks passed
bjoerge added a commit that referenced this pull request Oct 23, 2024
* next:
  v3.62.0
  fix: warnings on React 19 (#7654)
  fix(deps): update dependency @sanity/ui to ^2.8.10 (#7652)
  fix(deps): update dependency @sanity/ui to ^2.8.10 (#7653)
  chore(deps): update dependency @sanity/visual-editing to v2.2.2 (#7651)
  fix(core): inherit readOnly state from ancestors in copyPaste function (#7643)
  fix(deps): update dependency @sanity/mutate to ^0.10.1 (#7650)
  fix(deps): Update dev-non-major (#7649)
  fix(deps): update dependency @sanity/presentation to v1.17.3 (#7648)
  fix(deps): update dependency @portabletext/editor to ^1.1.5 (#7638)
  fix(deps): update dependency @sanity/presentation to v1.17.2 (#7647)
  fix(deps): Update dev-non-major (#7646)
  fix(deps): update dependency @sanity/presentation to v1.17.1 (#7645)
  chore(deps): update dependency @sanity/preview-url-secret to v2 (#7641)
  fix(deps): update dependency @sanity/presentation to v1.17.0 (#7640)
  fix(deps): Update dev-non-major (#7639)
  chore(cli): update help copy for deploy command (#7624)
  chore(test-studio): add asset types to structure navigation (#7631)
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