-
Notifications
You must be signed in to change notification settings - Fork 7
ci: remove upload documents from matrix job #959
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
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
Moves the document upload logic out of the per-matrix build job into a dedicated single-run job to prevent multiple redundant uploads on releases.
- Removes document upload steps from the matrix job.
- Adds a new upload-documents job that depends on build-push-test and performs a single release asset upload.
✅MegaLinter analysis: Success
See detailed reports in MegaLinter artifacts Your project could benefit from a custom flavor, which would allow you to run only the linters you need, and thus improve runtime performances. (Skip this info by defining
|
📦 Container Size AnalysisNote Comparing 📈 Size Comparison Table
|
📦 Container Size AnalysisNote Comparing 📈 Size Comparison Table
|
|
Pull Request Report (#959)Static measures
Time related measures
Status check related measures
|
|
🎉 Hooray! The changes in this pull request went live with the release of v6.5.1 🎉 |




🚀 Hey, I have created a Pull Request
Description of changes
Document uploads as part of a release have been wrongfully added to the matrix job that runs for every container. Documents should only be uploaded once, so moved document upload to a separate job.
✔️ Checklist