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

feat: adds mini preview on each deploy #921

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

Conversation

tnolet
Copy link
Member

@tnolet tnolet commented Jan 9, 2024

I hereby confirm that I followed the code guidelines found at engineering guidelines

Affected Components

  • CLI
  • Create CLI
  • Test
  • Docs
  • Examples
  • Other

Notes for the Reviewer

  • Adds a small, extra preview to each deploy command.
  • Refactors the preview fetching to a dedicated method and adds a type. Adds a counter and aligns indentation.
  • Reshuffles some copy and format on the full --preview command

Examples

Mini preview example

timnolet@MacBook-Pro-van-Tim beige-gazelle % npx checkly deploy          
Parsing your project... done

Deploy preview:

 - 1 to create, 0 to delete

 For a full preview, run: npx checkly deploy --preview

? You are about to deploy your project "Acme webapp" to account "Acme Production". Do you want to continue? › (y/N)

CleanShot 2024-01-09 at 18 03 13

Full preview example:

timnolet@MacBook-Pro-van-Tim beige-gazelle % npx checkly deploy --preview
Parsing your project... done
Creating (1):
 ApiCheck: checkly-api-123

Updating or leaving unchanged (4):
 BrowserCheck: __checks__/docs.spec.ts
 ApiCheck: checkly-api-1
 ApiCheck: checkly-api-2
 BrowserCheck: checkly-homepage

Skipping because of testOnly (1):
 ApiCheck: checkly-api-12

CleanShot 2024-01-09 at 18 03 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.

1 participant