Repository navigation
👷 ci: run nightly checks in the overnight window - #8
Merged
Merged
Conversation
Scheduled checks started at fixed hours shared with the rest of the org, so their runs competed for the same runner pools. Cron runs in UTC and ignores daylight saving, so the slot 07:00 to 11:59 UTC falls between 11pm and 5am Pacific under both PST and PDT. Each repo takes its own slot, eight minutes apart, so scheduled runs start one at a time instead of in a burst.
Mixed activityActivity patterns show a mix of organic and automated signals. Evidence
Last 5 PRs: This is an automated analysis by AgentScan |
for more information, see https://pre-commit.ci
The bot autofix rewrote a noqa comment in the plugin source. It is not part of the cron change, so this reverts it to keep the PR to the schedule line.
for more information, see https://pre-commit.ci
webknjaz
added a commit
that referenced
this pull request
Nov 14, 2026
A pre-commit.ci autofix in #8 rewrote `# noqa: ARG001` as Ruff's `# ruff: ignore[unused-function-argument]`. flake8 can't read that form, and the longer comment pushed the line past the length limit it enforces. Ruff no longer rewrites `noqa` comments, so the original one is back. Assisted-by: LLM
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The scheduled checks in this repo start at fixed hours that overlap the rest of the org's nightly jobs, and those jobs share the same runner pools. Moving this run into the overnight window, 11pm to 5am Pacific, keeps it out of the working day and away from the other jobs that start at the same time. ⏰
Cron expressions are in UTC and ignore daylight saving. The new slot sits between 07:00 and 11:59 UTC, which is 11pm to 3:59am Pacific during PST and midnight to 4:59am during PDT, so the run stays in the window through both seasons.
Each repo in the org has its own slot, and the slots are spaced eight minutes apart. Scheduled runs start one at a time instead of in a burst, so the runner pools do not see a spike at a single minute.
Only the
cronexpression of the scheduled workflow changes. The jobs, their inputs and manual triggers stay the same. The nightly run starts at a different UTC time after this merges.