cli icon indicating copy to clipboard operation
cli copied to clipboard

Use shop.dev, not my.shop.dev, for shopify

Open bbraschi opened this issue 7 months ago β€’ 2 comments

Wont's ship - would need instead an update of the whole package in vendor

bbraschi avatar May 08 '25 20:05 bbraschi

We detected some changes at packages/*/src and there are no updates in the .changeset. If the changes are user-facing, run pnpm changeset add to track your changes and include them in the next release CHANGELOG.

[!CAUTION] DO NOT create changesets for features which you do not wish to be included in the public changelog of the next CLI release.

github-actions[bot] avatar May 08 '25 20:05 github-actions[bot]

Coverage report

St.:grey_question:
Category Percentage Covered / Total
🟑 Statements
76.62% (-0.01% πŸ”»)
9588/12514
🟑 Branches
71.89% (-0.07% πŸ”»)
4729/6578
🟑 Functions 76.56% 2485/3246
🟑 Lines
77.13% (-0.01% πŸ”»)
9061/11748
Show files with reduced coverage πŸ”»
St.:grey_question:
File Statements Branches Functions Lines
🟒
... / link.ts
96.55%
92.42% (-0.43% πŸ”»)
100% 96.39%
🟒
... / mkcert.ts
93.33% (-3.28% πŸ”»)
76% (-7.87% πŸ”»)
100%
93.02% (-5.22% πŸ”»)
πŸ”΄
... / app-management-client.ts
41.78% (-0.4% πŸ”»)
36.75% (-2.59% πŸ”»)
40%
40.37% (-0.44% πŸ”»)
🟒
... / ConcurrentOutput.tsx
98.36% (-1.64% πŸ”»)
88% (-4% πŸ”»)
100%
98.33% (-1.67% πŸ”»)
🟒
... / version.ts
88.89% (-0.58% πŸ”»)
90% 100%
87.5% (-0.74% πŸ”»)

Test suite run success

2252 tests passing in 981 suites.

Report generated by πŸ§ͺjest coverage report action from 6a9ac275b16f9f6d72125b033c40b63ab54d2690

github-actions[bot] avatar May 08 '25 20:05 github-actions[bot]

This PR seems inactive. If it's still relevant, please add a comment saying so. Otherwise, take no action. β†’ If there's no activity within a week, then a bot will automatically close this. Thanks for helping to improve Shopify's dev tooling and experience.

github-actions[bot] avatar Jun 22 '25 03:06 github-actions[bot]

Not yet ready to be discarded

bbraschi avatar Jun 23 '25 18:06 bbraschi

The original dev-server package still has the my.shop.dev value. Is there a plan to update it there?

isaacroldan avatar Jul 04 '25 12:07 isaacroldan

@isaacroldan I am not sure and I did not spend the time to bug the team in charge of DevServer because the CLI started to work, [for me], out of the box, without this patch.

bbraschi avatar Jul 04 '25 15:07 bbraschi

This PR seems inactive. If it's still relevant, please add a comment saying so. Otherwise, take no action. β†’ If there's no activity within a week, then a bot will automatically close this. Thanks for helping to improve Shopify's dev tooling and experience.

github-actions[bot] avatar Aug 04 '25 04:08 github-actions[bot]

Closed as I did not experience problems in local dev recently

bbraschi avatar Aug 04 '25 12:08 bbraschi