Skip to content

open issues when scheduled linkcheck fails - #2993

Open
jyn514 wants to merge 1 commit into
rust-lang:mainfrom
jyn514:track-broken-links
Open

open issues when scheduled linkcheck fails#2993
jyn514 wants to merge 1 commit into
rust-lang:mainfrom
jyn514:track-broken-links

Conversation

@jyn514

@jyn514 jyn514 commented Aug 29, 2026

Copy link
Copy Markdown
Member

fixes #1956.
see jyn514#2 for example issues/comments it creates.
see https://github.com/rust-lang/rustc-dev-guide/actions/runs/33242644181/job/99074488060 for an example of the failures it'll link.

disclosure: most of the GHA crap here was generated by an LLM; it also found the GHA library, with prompting. i wrote the comments and PR description, and tested manually on my fork that it works.

i used an Action mostly because it seemed easy. i could port this to a Rust crate if you think it's better, seems mildly annoying to maintain but i don't feel strongly.

r? @jieyouxu

@rustbot rustbot added the S-waiting-on-review Status: this PR is waiting for a reviewer to verify its content label Aug 29, 2026

@jieyouxu jieyouxu left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Concept wise this seems like a good idea so broken links actually show up somewhere. Unsure about the third-party action picked since it doesn't really see much usage?

^ I don't care if the impl is GHA or Rust crate since I don't think it really matters, it's more about the action used specifically

View changes since this review

Comment thread .github/workflows/ci.yml
Comment thread .github/workflows/ci.yml Outdated

- name: Open an issue or comment on an existing one
id: report
uses: drivendataorg/failed-build-issue-action@940568ce50eeef3f31a920614cefcc6102d2f930 # v1.3.0

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hm. I don't love a dependency on a not well-known GHA action (https://github.com/drivendataorg/failed-build-issue-action) 🤔

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Unsure about the third-party action picked since it doesn't really see much usage?

yeah, it's either this or write our own thing on top of the gh CLI, unfortunately there don't seem to be a lot of libraries for this. i can rewrite this on gh if you prefer.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think it's probably preferrable if we could do it based on gh 👍

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

ok, rewrote it around that. i'll be honest the code is a big heaping pile of LLM bullshit, but it's not any worse than the GHA YAML.

- identify issues by label and title
- reuse an existing issue if it already exists
- use a bash script around `gh` so we don't add unvetted dependencies
- add tests for the bash script using a fake `gh` mock
@jyn514
jyn514 force-pushed the track-broken-links branch from 2df922a to e90126e Compare August 29, 2026 14:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-review Status: this PR is waiting for a reviewer to verify its content

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Automated Dead Link Detection

3 participants