-
Notifications
You must be signed in to change notification settings - Fork 223
Fix the default environments infrastructure so it doesn’t fail when running commands that don’t require authentication #6795
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
base: main
Are you sure you want to change the base?
Conversation
…unning commands that don’t require authentication
|
/snapit |
|
🫰✨ Thanks @karreiro! Your snapshot has been published to npm. Test the snapshot by installing your package globally: npm i -g --@shopify:registry=https://registry.npmjs.org @shopify/cli@0.0.0-snapshot-20260123102048Caution After installing, validate the version by running just |
Coverage report
Show new covered files 🐣
Show files with reduced coverage 🔻
Test suite run success3691 tests passing in 1431 suites. Report generated by 🧪jest coverage report action from 947025f |
WHY are these changes introduced?
shopify theme checkcurrently fails when the default environment is set inshopify.theme.toml.WHAT is this pull request doing?
This PR updates the multi-env infrastructure so it doesn’t fail when
--storeisn’t included in the list of required flags.How to test your changes?
shopify.theme.tomlfileshopify theme checkPost-release steps
N/A
Measuring impact
How do we know this change was effective? Please choose one:
Checklist