Skip to content

Bump link-check to fix multiple email link handing#511

Merged
BaseMax merged 2 commits intotcort:masterfrom
DanielYang59:bump-link-check-multi-email
Sep 22, 2025
Merged

Bump link-check to fix multiple email link handing#511
BaseMax merged 2 commits intotcort:masterfrom
DanielYang59:bump-link-check-multi-email

Conversation

@DanielYang59
Copy link
Contributor

fix #378

@DanielYang59 DanielYang59 changed the title Bump link-check to handle multi email link handing Bump link-check to fix multiple email link handing Sep 18, 2025
@BaseMax
Copy link
Collaborator

BaseMax commented Sep 18, 2025

Many thanks for your help. Approved to keep running tests and CICDs.

@BaseMax
Copy link
Collaborator

BaseMax commented Sep 18, 2025

It seems we have one failed CICD, would you please check and fix it? It can be from recent days not yours but it can be good if you can fix it as well.

https://github.com/tcort/markdown-link-check/actions/runs/17825862624/job/50680043041?pr=511

Many thanks.

@MikeMcC399
Copy link
Contributor

@BaseMax

The CI failure is in Node.js 18 which reached end-of-life on Apr 30, 2025 already.

I would suggest to update the tested Node.js versions in ci.yml in a separate PR

20, 22 & 24 are the currently supported release lines of Node.js

@DanielYang59
Copy link
Contributor Author

No problem at all, a bit unsure why it failed (Node 18 also passed on windows and macos) but looks like it might be related to my change, or perhaps can you rerun that work flow to see if it's intermittent?

  1. markdown-link-check
    should check the links in sample.md:
    Error: Timeout of 5000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (/home/runner/work/markdown-link-check/markdown-link-check/test/markdown-link-check.test.js)
    at listOnTimeout (node:internal/timers:569:17)
    at process.processTimers (node:internal/timers:512:7)

@MikeMcC399
Copy link
Contributor

@BaseMax

@MikeMcC399
Copy link
Contributor

@BaseMax

I suggest also re-running the failed test, as it is a timeout error, and this just may be flake and it could pass if you try again.

If it does not pass, then I would suggest reviewing and merging the CI update to the Node.js versions, as I don't think that it is worthwhile to spend time to fix an issue on Node.js 18 that is already past its end-of-life milestone.

@BaseMax BaseMax merged commit d609d79 into tcort:master Sep 22, 2025
11 of 12 checks passed
@BaseMax
Copy link
Collaborator

BaseMax commented Sep 22, 2025

Many thanks. I updated CI, lets double check again.

@DanielYang59 DanielYang59 deleted the bump-link-check-multi-email branch September 22, 2025 17:39
@DanielYang59
Copy link
Contributor Author

no problem ;)

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.

mailto: link with multiple emails isn't handled correctly

3 participants