Skip to content

gitignore linkcheck#2252

Merged
ehuss merged 1 commit intorust-lang:masterfrom
tlyu:ignore-linkcheck
May 4, 2026
Merged

gitignore linkcheck#2252
ehuss merged 1 commit intorust-lang:masterfrom
tlyu:ignore-linkcheck

Conversation

@tlyu
Copy link
Copy Markdown
Contributor

@tlyu tlyu commented May 2, 2026

Ignore linkcheck.sh, which is downloaded by the linkcheck test.

Ignore linkcheck.sh, which is downloaded by the linkcheck test.
@rustbot rustbot added the S-waiting-on-review Status: The marked PR is awaiting review from a maintainer label May 2, 2026
Copy link
Copy Markdown
Contributor

@DanielEScherzer DanielEScherzer left a comment

Choose a reason for hiding this comment

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

Makes sense to me, that file is not currently in the repo so should not be added unintentionally

View changes since this review

Copy link
Copy Markdown
Contributor

@ehuss ehuss left a comment

Choose a reason for hiding this comment

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

Thanks! I'm going to go ahead and approve for now. But I feel like there could be a better longer-term solution that doesn't leave the file lying around. Perhaps the xtask should just automatically delete it?

Unfortunately deleting it would interrupt a workflow of doing iterative linkcheck testing, which can be much faster. Maybe that could somehow be natively handled in the xtask, and store the script somewhere out of the way? I just don't know where that would be.

View changes since this review

@ehuss ehuss added this pull request to the merge queue May 4, 2026
Merged via the queue into rust-lang:master with commit e6b3010 May 4, 2026
6 checks passed
@rustbot rustbot removed the S-waiting-on-review Status: The marked PR is awaiting review from a maintainer label May 4, 2026
@tlyu tlyu deleted the ignore-linkcheck branch May 5, 2026 19:49
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.

4 participants