Add broken link checker on PR#360
Open
HrisShterev wants to merge 62 commits intocompiler-research:masterfrom
Open
Add broken link checker on PR#360HrisShterev wants to merge 62 commits intocompiler-research:masterfrom
HrisShterev wants to merge 62 commits intocompiler-research:masterfrom
Conversation
Add broken link checker to workflow
Updated GitHub Actions workflow to use newer versions of checkout and setup-python actions, and modified link checking to use the local server instead of a live URL.
Updated Ruby setup action to version 3 and removed URL ignore rule.
Updated the workflow to build and check links using MkDocs and linkchecker instead of HTMLProofer.
Added a step to check for broken links in the Jekyll build process using HTML-Proofer.
Add link checker #2
Updated broken link check to print all output and exit with code.
link checker fix
Added step to install html-proofer for link checking.
Install html-proofer in Jekyll workflow
Updated the link checking step to provide more detailed output.
Replaced html-proofer installation with Ruby setup step and updated link checking command.
Add link checker #2
Removed the html-proofer gem from the jekyll_plugins group.
Removed Ruby setup and broken link check steps, added htmlproofer for link validation.
Add link checker #2
Updated the htmlproofer step to check for broken links with additional options for ignoring certain errors.
Modify htmlproofer step to check for broken links
Added extra_args to the link-checking step for quiet mode.
Enhance link-checking with extra_args option
Refactor extra_args for better readability and include ignored options.
Update jekyll.yml to improve extra_args formatting
Refactor Jekyll workflow configuration
Update alt text handling in Jekyll workflow
Updated HTMLProofer configuration to check internal links only and removed external link checks.
Add link checker #2
Update link checking configuration in jekyll.yml
Update HTML Proofer settings in jekyll.yml
Replaced htmlproofer-action with a Docker command for link checking.
Update link checking method in Jekyll workflow
Updated HTML Proofer options to disable specific checks.
Modify HTML Proofer checks in jekyll.yml
Updated HTML Proofer options to include new flags for missing alt attributes and empty alt attributes.
Modify HTML Proofer options in jekyll.yml
Removed several options from the html-proofer command.
Simplify html-proofer command options
Update jekyll.yml
Update jekyll.yml
Update jekyll.yml
Add pull_request trigger to Jekyll workflow
vgvassilev
reviewed
Feb 1, 2026
.github/workflows/jekyll.yml
Outdated
| name: Deploy Jekyll site to Pages | ||
|
|
||
| on: | ||
| pull_request: |
Contributor
There was a problem hiding this comment.
I realize that we should have a separate file that works for pull requests. That’s for making the website live.
Removed the broken link check step from the Jekyll deployment workflow.
Contributor
|
This looks good. Now, we need to do a couple of things for this to land:
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.