Skip to content
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

chore(website): Update link checker to use Lychee #5337

Merged
merged 6 commits into from
Feb 19, 2025

Conversation

X-Guardian
Copy link
Contributor

@X-Guardian X-Guardian commented Feb 16, 2025

what

Update the GitHub workflow link checker from Muffet to Lychee.

Also renamed the GitHub workflow job from link-check to website-check to better reflect what the job does (Checks the links, builds the website and runs website e2e tests). The GitHub branch protection required status check will need updating from Website Link Check to Website Check to merge this PR.

why

Lychee is more actively developed that Muffet, and has a supported GitHub Action. It also allows the website source files to be scanned.

@X-Guardian X-Guardian requested review from a team as code owners February 16, 2025 15:20
@X-Guardian X-Guardian requested review from GenPage, lukemassa and nitrocode and removed request for a team February 16, 2025 15:20
@github-actions github-actions bot added docs Documentation blog labels Feb 16, 2025
@X-Guardian X-Guardian marked this pull request as draft February 16, 2025 15:21
@X-Guardian
Copy link
Contributor Author

You are reviewing a draft PR @chenrui333. Please review #5335 which has the details of these changes.

@chenrui333
Copy link
Member

okay, left the same comments there 👍

@X-Guardian X-Guardian marked this pull request as ready for review February 19, 2025 11:10
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Feb 19, 2025
@chenrui333 chenrui333 changed the title chore: (website): Update link checker to use Lychee chore(website): Update link checker to use Lychee Feb 19, 2025
@chenrui333 chenrui333 merged commit e8df55e into runatlantis:main Feb 19, 2025
35 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blog docs Documentation github-actions lgtm This PR has been approved by a maintainer website
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants