Repository navigation
Add --no-input to all commands - #8407
Conversation
|
/snapit |
|
🫰✨ Thanks @gonzaloriestra! Your snapshot has been published to npm. Test the snapshot by installing your package globally: pnpm i -g --@shopify:registry=https://registry.npmjs.org @shopify/cli@0.0.0-snapshot-20260831073953Caution After installing, validate the version by running |
3d4a0ca to
9a6f009
Compare
|
/snapit |
|
🫰✨ Thanks @gonzaloriestra! Your snapshot has been published to npm. Test the snapshot by installing your package globally: pnpm i -g --@shopify:registry=https://registry.npmjs.org @shopify/cli@0.0.0-snapshot-20260831104246Caution After installing, validate the version by running |
9a6f009 to
657cff4
Compare
dmerand
left a comment
There was a problem hiding this comment.
Tested against theme/dev/auth and it's working as desired 👍🏼 .
657cff4 to
76d7ab1
Compare
76d7ab1 to
2c19097
Compare
2c19097 to
2322c04
Compare
Differences in type declarationsWe detected differences in the type declarations generated by Typescript for this branch compared to the baseline ('main' branch). Please, review them to ensure they are backward-compatible. Here are some important things to keep in mind:
New type declarationspackages/cli-kit/dist/public/node/no-input.d.tsexport declare const noInputFlag: {
'no-input': import("@oclif/core/interfaces").BooleanFlag<boolean>;
};
/**
* Returns whether the current process explicitly disabled user input.
*
* @param argv - Process arguments to inspect.
* @param environment - Environment variables to inspect.
* @returns True when `--no-input` is active.
*/
export declare function isInputDisabled(argv?: string[], environment?: NodeJS.ProcessEnv): boolean;
Existing type declarationspackages/cli-kit/dist/public/node/cli.d.ts@@ -35,6 +35,7 @@ export declare function runCreateCLI(options: RunCLIOptions, launchCLI?: (option
export declare const globalFlags: {
'no-color': import("@oclif/core/interfaces").BooleanFlag<boolean>;
verbose: import("@oclif/core/interfaces").BooleanFlag<boolean>;
+ 'no-input': import("@oclif/core/interfaces").BooleanFlag<boolean>;
};
export declare const jsonFlag: {
json: import("@oclif/core/interfaces").BooleanFlag<boolean>;
|
karreiro
left a comment
There was a problem hiding this comment.
Thank you, @gonzaloriestra!
WHY are these changes introduced?
Closes shop/issues-develop#23696
Scripts and agents need to disable interactive behavior independently from output format.
WHAT is this pull request doing?
--no-inputalongside the existing global flags.How to test your changes?
shopify app deploy --helpand confirm--no-inputis listed.shopify app deploy --no-inputand confirm it shows the expected error.Checklist