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

Fix links, add CI workflows #113

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

qmonnet
Copy link
Collaborator

@qmonnet qmonnet commented Nov 20, 2024

  • package.json: Update dependencies
  • ci: Add CI workflows
  • readme.md: Fix (some) broken links

Let's use up-to-date linters and link checkers.

Signed-off-by: Quentin Monnet <[email protected]>
Reuse the configuration we have for remark to create two CI workflows:

- One runs on Pull Requests to make sure that the formatting of the
  README.md file remains OK.
- The other one runs manually, to check that all links are still valid.
  We could make it run as a cron job, but to be honest that would just
  waste resources because it's useless if nobody checks the results
  periodically to clean up invalid links.

Signed-off-by: Quentin Monnet <[email protected]>
Let's fix the links that are not longer valid. When possible, update
with the new link; otherwise, remove the associated entry.

After this change, the link checker still stumbles on a few links, but
they are valid in the browser, and I see no obvious way to fix them. I
suspect some CDNs reject bots or automated link checks and we're hitting
this.

Signed-off-by: Quentin Monnet <[email protected]>
@qmonnet
Copy link
Collaborator Author

qmonnet commented Nov 20, 2024

@zoidyzoidzoid Would you mind enabling GitHub Actions on the repo, please?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant