-
Notifications
You must be signed in to change notification settings - Fork 275
feat(cli): generate config files for local testing #14962
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: dev
Are you sure you want to change the base?
Conversation
| const projectPath = inputs[QuestionNames.ProjectPath] as string; | ||
| const includePlayground = inputs["include-playground"]; | ||
| const includeLocalDebug = inputs["include-local"]; | ||
| const includeRemoteDeploy = inputs["include-remote"]; |
Check notice
Code scanning / CodeQL
Unused variable, import, function or class Note
Show autofix suggestion
Hide autofix suggestion
Copilot Autofix
AI 7 days ago
To fix the unused variable problem for includeRemoteDeploy in generateConfigFiles, we should remove its declaration from the list of variables assigned from inputs. This change should be made specifically at line 23. No additional methods, definitions, or imports are needed, and the functionality of the code will remain the same as the variable was not used elsewhere.
| @@ -20,7 +20,6 @@ | ||
| const projectPath = inputs[QuestionNames.ProjectPath] as string; | ||
| const includePlayground = inputs["include-playground"]; | ||
| const includeLocalDebug = inputs["include-local"]; | ||
| const includeRemoteDeploy = inputs["include-remote"]; | ||
| const programmingLanguage = inputs["programming-language"] as string; | ||
|
|
||
| const appManifest = await AppManifestUtils.readTeamsManifest( |
feature flag: TEAMSFX_GENERATE_CONFIG_FILES
new CLI command:
atk init.Telemetry event: