Skip to content

add single-crate consistency check, using the sparse index - #3479

Merged
syphar merged 2 commits into
rust-lang:mainfrom
syphar:new-consistency
Aug 31, 2026
Merged

add single-crate consistency check, using the sparse index#3479
syphar merged 2 commits into
rust-lang:mainfrom
syphar:new-consistency

Conversation

@syphar

@syphar syphar commented Aug 27, 2026

Copy link
Copy Markdown
Member

thinking about the future where the git index might be only updated once an hours, we need another source for the consistency check.

So with this, when the db is outdated for a single crate, we can just sync that.

I imagine that we could also run a scheduler similar to the repo stats.

the removed queue.attempt was obsolete code that's not needed any more

( test failure is unrelated)

@syphar syphar self-assigned this Aug 27, 2026
@github-actions github-actions Bot added the S-waiting-on-review Status: This pull request has been implemented and needs to be reviewed label Aug 27, 2026
@syphar syphar changed the title confs add single-crate consistency check, using the sparse index Aug 27, 2026
@syphar
syphar marked this pull request as ready for review August 27, 2026 08:40
@syphar
syphar requested a review from a team as a code owner August 27, 2026 08:40
@syphar

syphar commented Aug 27, 2026

Copy link
Copy Markdown
Member Author

r? @GuillaumeGomez

@rustbot rustbot assigned GuillaumeGomez and unassigned syphar Aug 27, 2026
@GuillaumeGomez

Copy link
Copy Markdown
Member

Failed CI.

@syphar

syphar commented Aug 27, 2026

Copy link
Copy Markdown
Member Author

Failed CI.

that test failure is unrelated

Comment thread crates/bin/docs_rs_watcher/src/consistency/db.rs Outdated
@syphar
syphar requested a review from GuillaumeGomez August 31, 2026 09:41
@GuillaumeGomez

Copy link
Copy Markdown
Member

Thanks!

@syphar
syphar merged commit 7af11e8 into rust-lang:main Aug 31, 2026
11 checks passed
@syphar
syphar deleted the new-consistency branch August 31, 2026 09:43
@github-actions github-actions Bot added S-waiting-on-deploy This PR is ready to be merged, but is waiting for an admin to have time to deploy it and removed S-waiting-on-review Status: This pull request has been implemented and needs to be reviewed labels Aug 31, 2026
@syphar syphar removed the S-waiting-on-deploy This PR is ready to be merged, but is waiting for an admin to have time to deploy it label Sep 1, 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