Skip to content

Use flint for link checking#546

Closed
zeitlinger wants to merge 4 commits intoopen-telemetry:mainfrom
zeitlinger:use-flint-for-link-checking
Closed

Use flint for link checking#546
zeitlinger wants to merge 4 commits intoopen-telemetry:mainfrom
zeitlinger:use-flint-for-link-checking

Conversation

@zeitlinger
Copy link
Copy Markdown
Member

@zeitlinger zeitlinger commented Feb 25, 2026

Blocked by grafana/flint#139

Summary

  • Switch from 3 separate contrib-hosted link check scripts to the consolidated flint lint:links task
  • SHA-pin the flint URL with Renovate custom manager for automatic updates
  • Remove lychee#1729 workaround (now handled by flint at runtime)

Test plan

  • CI link check passes on this PR

Signed-off-by: Gregor Zeitlinger <gregor.zeitlinger@grafana.com>
@zeitlinger zeitlinger marked this pull request as draft April 7, 2026 05:54
Copy link
Copy Markdown
Contributor

@codeboten codeboten left a comment

Choose a reason for hiding this comment

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

Is this ready for review? CI has been failing to check links for a while

@zeitlinger
Copy link
Copy Markdown
Member Author

Is this ready for review? CI has been failing to check links for a while

I'm just updating to the new flint version - and will update this pr when it's ready.

Copy link
Copy Markdown
Member Author

zeitlinger commented Apr 22, 2026

Superseded by a fresh v0.20.3 Rust-flint PR based on current main: #600.

I left #546 alone rather than force-rewriting the old pre-release shell-task branch. The replacement keeps the scope small and uses the released github:grafana/flint = "0.20.3" install path.

@zeitlinger zeitlinger closed this Apr 22, 2026
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