Skip to content

consistency check: ignore subseconds for release-time compare - #3509

Merged
GuillaumeGomez merged 1 commit into
rust-lang:mainfrom
syphar:sync
Sep 8, 2026
Merged

GuillaumeGomez merged 1 commit into
rust-lang:mainfrom
syphar:sync

Conversation

@syphar

@syphar syphar commented Sep 8, 2026

Copy link
Copy Markdown
Member

when dry-running the consistency check, there were release-time differences shown for all releases.

Reason:

  • crates.io API and our DB has sub-seconds for release-time: 2022-04-15 08:49:30.157869+00,
  • the sparse index has only seconds: 2022-04-15T08:49:30Z

Here we just ignore sub-seconds when comparing.

@syphar syphar self-assigned this Sep 8, 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 Sep 8, 2026
@syphar
syphar marked this pull request as ready for review September 8, 2026 08:50
@syphar
syphar requested a review from a team as a code owner September 8, 2026 08:50
@syphar

syphar commented Sep 8, 2026

Copy link
Copy Markdown
Member Author

r? @GuillaumeGomez

@rustbot rustbot assigned GuillaumeGomez and unassigned syphar Sep 8, 2026
@GuillaumeGomez

Copy link
Copy Markdown
Member

Thanks!

@GuillaumeGomez
GuillaumeGomez merged commit bbe8284 into rust-lang:main Sep 8, 2026
11 checks passed
@GuillaumeGomez
GuillaumeGomez deleted the sync branch September 8, 2026 09:39
@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 Sep 8, 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 9, 2026

This branch was previously deployed

1 inactive deployment
github-pages — 2769751a Deployed Sep 8, 2026 by GuillaumeGomez via deploy #253
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