Skip to content

.github: run GitHub workflow weekday#94

Merged
dongsupark merged 2 commits into
mainfrom
github-workflow-weekday
Jul 24, 2026
Merged

.github: run GitHub workflow weekday#94
dongsupark merged 2 commits into
mainfrom
github-workflow-weekday

Conversation

@dongsupark

Copy link
Copy Markdown
Member

Run security-triage workflow only weekdays, not during weekend, to reduce number of reports to be reviewed.

Set read permission for test, install, lint, to resolve code scanning alerts.

Copilot AI review requested due to automatic review settings July 24, 2026 10:30
@dongsupark
dongsupark requested review from a team and danielszot as code owners July 24, 2026 10:30

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adjusts GitHub Actions scheduling and token permissions to reduce weekend security-triage noise and align workflows with least-privilege GITHUB_TOKEN permissions.

Changes:

  • Restricts the security-triage scheduled workflow to run only Monday–Friday (cron: '0 6 * * 1-5').
  • Sets explicit permissions: contents: read for the tests, lint, and install workflows to satisfy read-only requirements/code scanning expectations.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
.github/workflows/tests.yml Adds explicit read-only contents permission for the test workflow token.
.github/workflows/security-triage.yml Updates the scheduled cron to run on weekdays only.
.github/workflows/lint.yml Adds explicit read-only contents permission for the lint workflow token.
.github/workflows/install.yml Adds explicit read-only contents permission for the install workflow token.

@tormath1 tormath1 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

It makes sense to me. Let's update the unit tests as well:

assert crons == ["0 6 * * *"]

Run security-triage workflow only weekdays, not during weekend, to
reduce number of reports to be reviewed.

Signed-off-by: Dongsu Park <dongsu@dpark.io>
Set read permission for test, install, lint, to resolve code scanning
alerts.

Signed-off-by: Dongsu Park <dongsu@dpark.io>
Copilot AI review requested due to automatic review settings July 24, 2026 11:00
@dongsupark
dongsupark force-pushed the github-workflow-weekday branch from 614d390 to bbecc24 Compare July 24, 2026 11:00

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated no new comments.

@dongsupark
dongsupark merged commit 40c0053 into main Jul 24, 2026
25 checks passed
@dongsupark
dongsupark deleted the github-workflow-weekday branch July 24, 2026 15:45
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.

3 participants