Skip to content

Add --yes flag to skills add command#137

Merged
johnpryan merged 1 commit into
flutter:mainfrom
ayanmajumdar-m:patch-2
May 27, 2026
Merged

Add --yes flag to skills add command#137
johnpryan merged 1 commit into
flutter:mainfrom
ayanmajumdar-m:patch-2

Conversation

@ayanmajumdar-m
Copy link
Copy Markdown
Contributor

This PR updates the installation command in the README by adding the --yes flag to the npx skills add command.

What Changed

Before:

npx skills add flutter/skills --skill '*' --agent universal

After:

npx skills add flutter/skills --skill '*' --agent universal --yes
Why

The --yes flag enables non-interactive execution by automatically confirming prompts during installation. This improves automation support and provides a smoother experience for users running the command in CI/CD pipelines, scripts, or headless environments.

Fixes
Fixes issue where the installation command pauses for manual confirmation during automated execution.
Pre-launch Checklist
I read the Contributor Guide and followed the process outlined there for submitting PRs.
I read the Tree Hygiene wiki page, which explains my responsibilities.
I read the Flutter Style Guide recently, and have followed its advice.
I signed the CLA.
I listed at least one issue that this PR fixes in the description above.
I updated/added relevant documentation (doc comments with ///).
This PR is test-exempt (documentation-only change).
All existing and new tests are passing.

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the README.md file to include the --yes flag in the example command for adding skills, which facilitates non-interactive execution. There were no review comments provided for this change, and I have no additional feedback to provide.

@johnpryan johnpryan merged commit 939b874 into flutter:main May 27, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants