- Notifications
You must be signed in to change notification settings - Fork899
Add Tool Name property for List Code Scanning Alerts tool#272
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
Uh oh!
There was an error while loading.Please reload this page.
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 pull request adds an optional tool_name parameter to the ListCodeScanningAlerts function to allow specifying the tool used for code scanning and updates the corresponding tests and documentation.
- Introduces tool_name parameter in code_scanning.go
- Updates tests in code_scanning_test.go to assert the new parameter
- Enhances documentation in README.md to include tool_name
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| pkg/github/code_scanning_test.go | Adds assertions and test cases for the newly added tool_name property in input schema. |
| pkg/github/code_scanning.go | Introduces and passes the optional tool_name parameter to the GitHub client API call. |
| README.md | Updates the documentation to include the tool_name parameter description. |
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.
Thank you for this.
614f226 intomainUh oh!
There was an error while loading.Please reload this page.
This pull request introduces a new optional parameter,
tool_name, to theListCodeScanningAlertsfunction and updates the relevant documentation and tests accordingly.Closes:#271