Skip to content

Conversation

@f-hollow
Copy link
Collaborator

Description

Once pr-check-links is run, GitHub CI started returning Error: Compiled regex exceeds size limit of 10485760 bytes..

Apparently, the file .lycheeignore contains too many entries causing lychee to construct an oversized regex.

I tried several solutions:

  • Tried using --exclude for links instead of .lycheeignore,however, the pipeline resulted in failure.
  • Tried other countless fixes invain

The only working solution appeared to be splitting link checking into several stages:

  • non-blog - all files in ./content except for ./content/blog/ and README.md
  • blog_0-9a-f - all files in ./content/blog/[0-9a-f]*/ folders
  • blog_g-z- all files in ./content/blog/[g-z]*/ folders

Related

Testing

Extensive testing was done in the testing environment.


Checklist

Before submitting a Pull Request, please ensure the following:

  • 🚨 This PR does not introduce breaking changes.
  • All CI checks (GH Actions) pass.
  • Documentation is updated as needed.
  • Tests are updated or added as necessary.
  • Code is well-commented, especially in complex areas.
  • Git history is clean — commits are squashed to the minimum necessary.

@f-hollow f-hollow force-pushed the ci/fix_link_checking branch 2 times, most recently from b47f857 to 053140f Compare March 12, 2025 10:50
@horw
Copy link
Collaborator

horw commented Mar 12, 2025

Maybe in the future, it makes sense to move to our own runner to avoid this limitation for the regex buffer on GitHub's side. I am not sure whether this issue will disappear or not, but we can try it in the future.

@f-hollow f-hollow force-pushed the ci/fix_link_checking branch 3 times, most recently from 0e2829b to b8ff729 Compare March 12, 2025 11:16
@f-hollow f-hollow force-pushed the ci/fix_link_checking branch from b8ff729 to d3a8f34 Compare March 12, 2025 11:21
@f-hollow f-hollow force-pushed the ci/fix_link_checking branch from bcb0657 to 0738e8a Compare March 13, 2025 04:19
@f-hollow f-hollow merged commit e611fd1 into espressif:main Mar 13, 2025
12 checks passed
@f-hollow f-hollow deleted the ci/fix_link_checking branch March 13, 2025 04:22
@f-hollow
Copy link
Collaborator Author

f-hollow commented Apr 1, 2025

Fixes #409

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.

2 participants