Skip to content

Remove ellipsis from preview store creation task#7913

Merged
alfonso-noriega merged 1 commit into
mainfrom
remove-preview-create-ellipsis
Jun 24, 2026
Merged

Remove ellipsis from preview store creation task#7913
alfonso-noriega merged 1 commit into
mainfrom
remove-preview-create-ellipsis

Conversation

@alfonso-noriega

@alfonso-noriega alfonso-noriega commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

WHY are these changes introduced?

Fixes #0000

WHAT is this pull request doing?

How to test your changes?

Post-release steps

Checklist

  • I've considered possible cross-platform impacts (Mac, Linux, Windows)
  • I've considered possible documentation changes
  • I've considered analytics changes to measure impact
  • The change is user-facing — I've identified the correct bump type (patch for bug fixes · minor for new features · major for breaking changes) and added a changeset with pnpm changeset add

Assisted-By: devx/c4031af8-816c-4935-976b-42b220f2f7bf

Copy link
Copy Markdown
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@github-actions github-actions Bot added the no-changelog This PR doesn't include a changeset entry. Is an internal only change not relevant to end users. label Jun 24, 2026
@alfonso-noriega alfonso-noriega marked this pull request as ready for review June 24, 2026 09:07
@alfonso-noriega alfonso-noriega requested a review from a team as a code owner June 24, 2026 09:07

const result = await renderSingleTask<CreatePreviewStoreResult>({
title: outputContent`Creating store`,
title: outputContent`Creating store`,

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is outputContent even necessary here?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

That is the standard cli approach to static messages:

  title: outputContent`Authenticating`                                                                                                                                                                                                                                     
  title: outputContent`Executing GraphQL operation`                                                                                                                                                                                                                        
  title: outputContent`Starting bulk operation`                                                                                                                                                                                                                            
  title: outputContent`Polling bulk operation`                                                                                                                                                                                                                             
  title: outputContent`Uploading bulk operation variables`                                                                                                                                                                                                                 
  title: outputContent`Loading application`                                                                                                                                                                                                                                
  title: outputContent`Loading stored store auth`                                                                                                                                                                                                                          

@alfonso-noriega alfonso-noriega added this pull request to the merge queue Jun 24, 2026
Merged via the queue into main with commit 6808329 Jun 24, 2026
27 of 29 checks passed
@alfonso-noriega alfonso-noriega deleted the remove-preview-create-ellipsis branch June 24, 2026 10:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-changelog This PR doesn't include a changeset entry. Is an internal only change not relevant to end users.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants