-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Restructure Release Drafter Flow #9345
Conversation
Contextual Security AnalysisAs DryRun Security performs checks, we’ll summarize them here. You can always dive into the detailed results in the section below for checks.
Chat with your AI-powered Security Buddy by typing Install and configure more repositories at DryRun Security |
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.
One small note but I agree in general
Co-authored-by: kiblik <[email protected]>
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.
Approved
It failed again :( https://github.com/DefectDojo/django-DefectDojo/actions/runs/7614278448/job/20736452711 |
Got very lucky finding this SO post. Seems to be the problem |
Got it working :) I think next weeks release will go off without a hitch now https://github.com/DefectDojo/django-DefectDojo/releases/tag/2.30.3 |
Nice. Thank you :) |
This PR updates the structure and flow of the release drafter process. Now that OAS schemas are appended to each release object, it might be a better approach to make the release drafting process less event driven. Here is what the PR proposes:
@kiblik I encountered a few errors during the release this morning, and had a few thoughts to improve this process as I was going though it. I have not tested this PR in any capacity (it is a bit difficult to do that outside of releases) but please looks it over and let me know your thoughts. Here is the failed run from today