Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR removes the automatic issue creation functionality from the link checker CI job while maintaining the link checking itself. The change simplifies the workflow by removing the step that automatically creates GitHub issues when broken links are detected.
- Removes the "Create Issues" step that automatically generates GitHub issues for link checker failures
- Maintains the link checking functionality and failure detection
- Keeps the manual failure step that exits with an error code when issues are found
Comments suppressed due to low confidence (1)
.github/workflows/main.yml:1
- The error message still references "the created issue" but no issue is being created anymore. This should be updated to reference checking the workflow logs or output instead.
name: Main
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
0cb15a2 to
7df6774
Compare
markurtz
left a comment
There was a problem hiding this comment.
I'd like to keep the link checking CI on main/nightly. This is primarily because links on README.md or other places that need absolute references will not work until they are landed on main. Secondarily, adds one less job to wait on for PRs.
Towards the above, I think a nightly cadence would work best so we're not overloading main
a562539 to
9e34efa
Compare
Feel free to override or close this |
9e34efa to
a095921
Compare
|
@markurtz I think this current approach is a good balance, depending on whether we want the build to fail on broken link errors or pass with warnings. |
a095921 to
8f88b56
Compare
|
Offering this as an alternative to removing the nightly link checker. Instead of opening a new issue each time, we just have a single issue which we add comments to. |
49383b9 to
25f7a86
Compare
|
I'm going to stay in my lane on this one. This PR now just fixes some broken links. |
922e739 to
8ba7dea
Compare
Signed-off-by: Aidan Reilly <aireilly@redhat.com>
8ba7dea to
b384d6a
Compare
* Fixed lychee errors and added /reference/speculators to.lycheeignore.* Removed the auto-issue job, too noisy and not helpful* Moved the link check job to run on PR CI jobs, not main nightlyFix broken link errors.
Note, CI won't complete for PRs from forks, not sure if this is intended 🤷