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

Add GitHub App event security_advisory #2799

Merged
merged 3 commits into from
Jun 6, 2023
Merged

Add GitHub App event security_advisory #2799

merged 3 commits into from
Jun 6, 2023

Conversation

lukehinds
Copy link
Contributor

The GitHub App can transmit dependabot events for security_advisory

This PR adds the event, it is based off #2392, so hopefully the mapping is still the same.

Resolves: #2798

lukehinds added 3 commits June 6, 2023 15:34
The GitHub App can transmit dependabot events for security_advisory

This PR adds the event, it is based off #2392, so hopefully the mapping
is still the same.

Resolves: #2798

Signed-off-by: Luke Hinds <lukehinds@gmail.com>
Copy link
Collaborator

@gmlewis gmlewis left a comment

Choose a reason for hiding this comment

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

Thank you, @lukehinds !
LGTM.

Awaiting second LGTM+Approval from any other contributor to this repo before merging.

@gmlewis gmlewis added the NeedsReview PR is awaiting a review before merging. label Jun 6, 2023
@gmlewis gmlewis removed the NeedsReview PR is awaiting a review before merging. label Jun 6, 2023
@gmlewis
Copy link
Collaborator

gmlewis commented Jun 6, 2023

Thank you, @valbeat !
Merging.

@gmlewis gmlewis merged commit 5a9f8f7 into google:master Jun 6, 2023
@lukehinds
Copy link
Contributor Author

lukehinds commented Jun 6, 2023

Thanks @gmlewis , just noticed I missed the window for v53.0.0 , when would the next release be shipped?

@gmlewis
Copy link
Collaborator

gmlewis commented Jun 6, 2023

We usually make releases about every 4 weeks or so, but if there is urgency, we can be flexible and do more.
If you just want to use it in your own project, you can import a specific version of the repo... I think we have instructions how to do this in our README.md.

@lukehinds
Copy link
Contributor Author

lukehinds commented Jun 7, 2023

@gmlewis no urgency, will pin a commit for now and await the next release! thanks!

@gmlewis
Copy link
Collaborator

gmlewis commented Jun 12, 2023

@lukehinds - your changes are now here:
https://github.com/google/go-github/releases/tag/v53.1.0

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.

GitHub App event type security_advisory missing
3 participants