Removed redundant workflow definitions - #8642
Conversation
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Pull request overview
This PR removes multiple GitHub Actions workflow definitions (CI, release, reporting, automation) from this branch and adds a README in .github/workflows/ explaining that workflow changes should be made on main.
Changes:
- Deleted several workflow YAML files under
.github/workflows/(CI, release, reporting, automation). - Added
.github/workflows/README.mdto document expected workflow editing behavior for this branch.
Reviewed changes
Copilot reviewed 8 out of 8 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| .github/workflows/test-failures-report.yml | Removed test-failure aggregation/reporting workflow. |
| .github/workflows/sbt-dependency-graph.yml | Removed dependency graph submission workflow. |
| .github/workflows/release.yml | Removed release/publish workflow. |
| .github/workflows/mtags-release.yml | Removed mtags release workflow. |
| .github/workflows/close-inactive-needs-more-info-issues.yml | Removed stale-issue closing automation. |
| .github/workflows/ci.yml | Removed CI workflow for pushes/PRs to this branch. |
| .github/workflows/auto-approve.yml | Removed auto-approve workflow. |
| .github/workflows/README.md | Added documentation explaining where workflow changes should be made. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| Nearly all contributions in this repostory come as pull requests from forks. | ||
| For such pull requests, Github uses workflow definitions from the repository's main branch (i.e. `main`). | ||
|
|
||
| If you want to change an existing workflow or ad a new one, please submit a PR to the `main` branch; | ||
| making any changes on this branch will have no effect in practice. |
There was a problem hiding this comment.
Is that so? I'm pretty sure that if the PR comes from a fork, the workflow definitions are taken from the main branch, no matter the actual base of the PR.
If I am mistaken, please provide references to the documentation.
No description provided.