Conversation
This initial setup was derived from some preliminary testing of the tool in a throwaway repo containing some of the content in this repo. Based on that experience, we should expect multiple failures.
The azure link is to a registration page that contains editable fields, which might cause access failures. Tried changing it to just the landing page. Users should still be able to determine how to register to access software from that page.
Stefan has said that there will be a PR into develop for parmoo soon to account for a recent refactoring. This is a temporary fix to get the URL checker action passing. Ideally, this change will be in develop before they base their branch off of it and can adjust the text as needed.
Two were written with http instead of https. Lychees was ignoring these with the statement that they are unsupported. This will get rid of those statements and makes the links like the first azure link that I dealt with. Ultimately, they will be ignored.
It is using an old version of actions that rely on Node.js 20, which will be dropped soon. Might as well fix this with the URL checker as a single infrastructure hot fix.
DanielRPhillips
approved these changes
Mar 21, 2026
Member
DanielRPhillips
left a comment
There was a problem hiding this comment.
I have checked all the revised links in the new branch and all now go to the right place. Actions are passing.
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
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.
PR Self-review
Note that this tool is reporting that it is skipping the verification of several email addresses. While this seems OK, lychee should be checking email addresses by default.