- Notifications
You must be signed in to change notification settings - Fork2.7k
adding default toolset#1137
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 ourterms of service andprivacy 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
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Pull Request Overview
This PR changes the default toolset configuration from "all" to a specific set of core tools and updates related documentation to reflect the new default behavior.
- Replaces the single "all" toolset with explicit core toolsets: "context", "repos", "issues", "pull_requests"
- Removes the "// indirect" comment from a dependency in go.mod
- Updates flag description to remove reference to "enabling all"
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| pkg/github/tools.go | Updates DefaultTools from ["all"] to specific core toolsets |
| go.mod | Removes indirect comment from go-querystring dependency |
| cmd/github-mcp-server/main.go | Updates toolsets flag description to remove "all" reference |
Tip: Customize your code reviews with copilot-instructions.md.Create the file orlearn how to get started.
| github.com/go-viper/mapstructure/v2v2.4.0 | ||
| github.com/google/go-github/v71v71.0.0// indirect | ||
| github.com/google/go-querystringv1.1.0// indirect | ||
| github.com/google/go-querystringv1.1.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Removing the '// indirect' comment suggests this dependency is now being used directly, but there's no corresponding code change showing the direct usage. If this is indeed now a direct dependency, consider adding it to the non-indirect section of the dependencies or ensure the direct usage is properly reflected in the codebase.
| github.com/google/go-querystring v1.1.0 | |
| github.com/google/go-querystring v1.1.0// indirect |
Copilot uses AI. Check for mistakes.
Uh oh!
There was an error while loading.Please reload this page.
Closes:
Updated the help section to display all available toolset groupings