Skip to content

fix(ci): use anchored regex for lychee exclude-path patterns#547

Merged
mchmarny merged 3 commits intoNVIDIA:mainfrom
pdmack:pdmack/fern-ci-fix2
Apr 12, 2026
Merged

fix(ci): use anchored regex for lychee exclude-path patterns#547
mchmarny merged 3 commits intoNVIDIA:mainfrom
pdmack:pdmack/fern-ci-fix2

Conversation

@pdmack
Copy link
Copy Markdown
Contributor

@pdmack pdmack commented Apr 11, 2026

Summary

Use anchored regex patterns for lychee --exclude-path to prevent partial
directory name matches.

Motivation / Context

Bare path prefixes like docs/design would match docs/design-patterns or any
other path that starts with those characters. Anchored patterns
(^docs/design(/|$)) match only the exact directory and its contents.

Fixes: N/A
Related: N/A

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • Build/CI/tooling

Component(s) Affected

  • Other: Fern docs CI (.github/workflows/fern-docs-ci.yaml)

Implementation Notes

Changed --exclude-path 'docs/design' to --exclude-path '^docs/design(/|$)'
(and same for docs/plans, docs/conformance) so exclusions are anchored to
the exact directory, not any path that starts with those characters.

Testing

CI-only change — no make qualify applicable.

Risk Assessment

  • Low — Isolated change, well-tested, easy to revert

Rollout notes: N/A

Checklist

  • Tests pass locally (make test with -race)
  • Linter passes (make lint)
  • I did not skip/disable tests to make CI green
  • I added/updated tests for new functionality
  • I updated docs if user-facing behavior changed
  • Changes follow existing patterns in the codebase
  • Commits are cryptographically signed (git commit -S)

@pdmack pdmack requested a review from a team as a code owner April 11, 2026 21:17
@pdmack pdmack requested a review from a team as a code owner April 11, 2026 21:36
@github-actions
Copy link
Copy Markdown

@mchmarny mchmarny merged commit 56f20f6 into NVIDIA:main Apr 12, 2026
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants