Skip to content
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

Update code scanning alert #673

Merged
merged 9 commits into from
Aug 14, 2024
Merged

Update code scanning alert #673

merged 9 commits into from
Aug 14, 2024

Conversation

adrienpessu
Copy link
Contributor

This pull request introduces a new feature for updating code scanning alerts in a GitHub repository. The most important changes include adding a new module for handling updates, defining a new enum for alert states, and creating a test to verify the update functionality.

New Feature: Update Code Scanning Alerts

  • src/api/code_scannings.rs: Added the update module and the update method to the CodeScanningHandler to handle updating code scanning alerts. [1] [2]
  • src/api/code_scannings/update.rs: Implemented the UpdateCodeScanningBuilder struct, which includes methods for setting the state, dismissed reason, and dismissed comment of a code scanning alert, and a send method to execute the update.
  • src/params.rs: Added the AlertState enum to represent the state of a code scanning alert, with possible values Dismissed and Open.
  • tests/code_scanning_alert_update_test.rs: Created a test to verify that the update method correctly updates a code scanning alert and handles a successful response.

* Update Code Scanning alert
@adrienpessu adrienpessu requested a review from XAMPPRocky August 1, 2024 11:01
@XAMPPRocky
Copy link
Owner

Thank you for your PR!

@XAMPPRocky XAMPPRocky merged commit 8527406 into XAMPPRocky:main Aug 14, 2024
11 checks passed
@github-actions github-actions bot mentioned this pull request Aug 14, 2024
@github-actions github-actions bot mentioned this pull request Aug 30, 2024
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