Skip to content

Add link checking github action#1454

Merged
mm-kevcenteno merged 4 commits intomainfrom
kevin/lychee
Oct 16, 2025
Merged

Add link checking github action#1454
mm-kevcenteno merged 4 commits intomainfrom
kevin/lychee

Conversation

@mm-kevcenteno
Copy link
Contributor

  1. Adds Lychee
  2. Fixes broken links

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Oct 15, 2025

Deploying dev-site with  Cloudflare Pages  Cloudflare Pages

Latest commit: 187b916
Status: ✅  Deploy successful!
Preview URL: https://1463fe88.dev-site-4ua.pages.dev
Branch Preview URL: https://kevin-lychee.dev-site-4ua.pages.dev

View logs

Copy link
Contributor

@PatrickCroninMM PatrickCroninMM left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks great!

Copy link
Contributor

@PatrickCroninMM PatrickCroninMM left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great!

Comment on lines 64 to 68
- [Java](https://github.com/maxmind/minfraud-api-java/blob/main/src/main/java/com/maxmind/minfraud/request/Email.java#L414)
- [JavaScript](https://github.com/maxmind/minfraud-api-node/blob/main/src/request/email.ts#L313)
- [PHP](https://github.com/maxmind/minfraud-api-php/blob/main/src/MinFraud/Util.php#L284)
- [Python](https://github.com/maxmind/minfraud-api-python/blob/main/minfraud/request.py#L357)
- [Python](https://github.com/maxmind/minfraud-api-python/blob/main/src/minfraud/request.py#L375)
- [Ruby](https://github.com/maxmind/minfraud-api-ruby/blob/main/lib/minfraud/components/email.rb#L84)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I feel like it's a little ambitious to think that we'll make sure these links are updated. It might be better to use permalinks for GitHub links instead of the current main. That being said, this is pre-existing, so not related to your PR.

@mm-kevcenteno mm-kevcenteno merged commit 9bc344b into main Oct 16, 2025
15 checks passed
@mm-kevcenteno mm-kevcenteno deleted the kevin/lychee branch October 16, 2025 20:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants