Skip to content

Chore: [AEA-0000] - Depenadabot cooldown#2909

Merged
anthony-nhs merged 6 commits intomainfrom
depenadabot_cooldown
Mar 27, 2026
Merged

Chore: [AEA-0000] - Depenadabot cooldown#2909
anthony-nhs merged 6 commits intomainfrom
depenadabot_cooldown

Conversation

@anthony-nhs
Copy link
Copy Markdown
Contributor

Summary

  • Routine Change

Details

  • dependabot cooldown

Copilot AI review requested due to automatic review settings March 25, 2026 09:50
@github-actions
Copy link
Copy Markdown
Contributor

This PR is linked to a ticket in an NHS Digital JIRA Project. Here's a handy link to the ticket:

AEA-0000

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR introduces a Dependabot update “cooldown” setting and adds an automated GitHub Actions workflow to periodically update the devcontainer version.

Changes:

  • Add a new scheduled workflow to update the repository devcontainer version via a reusable workflow.
  • Add cooldown: default-days: 3 to all configured Dependabot ecosystems (pip, GitHub Actions, npm).

Reviewed changes

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

File Description
.github/workflows/update_dev_container_version.yml Adds a scheduled/manual workflow that calls a reusable workflow to update the devcontainer version.
.github/dependabot.yml Adds a 3-day cooldown configuration to each Dependabot update entry.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/dependabot.yml
Comment on lines +26 to +27
cooldown:
default-days: 3
Copy link

Copilot AI Mar 25, 2026

Choose a reason for hiding this comment

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

cooldown / default-days is not part of the standard Dependabot v2 configuration keys (keys are validated by GitHub). If this field isn't supported in your GitHub org/instance, it will cause Dependabot to reject the entire config and stop creating update PRs. Please verify support for cooldown or replace with an officially supported mechanism (e.g., adjust schedule, use groups, or rely on open-pull-requests-limit).

Copilot uses AI. Check for mistakes.
Comment thread .github/dependabot.yml
Comment on lines +39 to +40
cooldown:
default-days: 3
Copy link

Copilot AI Mar 25, 2026

Choose a reason for hiding this comment

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

cooldown / default-days is not part of the standard Dependabot v2 configuration keys (keys are validated by GitHub). If this field isn't supported in your GitHub org/instance, it will cause Dependabot to reject the entire config and stop creating update PRs. Please verify support for cooldown or replace with an officially supported mechanism (e.g., adjust schedule, use groups, or rely on open-pull-requests-limit).

Copilot uses AI. Check for mistakes.
Comment thread .github/dependabot.yml
Comment on lines +57 to +58
cooldown:
default-days: 3
Copy link

Copilot AI Mar 25, 2026

Choose a reason for hiding this comment

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

cooldown / default-days is not part of the standard Dependabot v2 configuration keys (keys are validated by GitHub). If this field isn't supported in your GitHub org/instance, it will cause Dependabot to reject the entire config and stop creating update PRs. Please verify support for cooldown or replace with an officially supported mechanism (e.g., adjust schedule, use groups, or rely on open-pull-requests-limit).

Copilot uses AI. Check for mistakes.
Comment on lines +1 to +10
name: Update Devcontainer Version

on:
workflow_dispatch:
schedule:
- cron: "0 18 * * 4"

jobs:
update_devcontainer_version:
uses: NHSDigital/eps-common-workflows/.github/workflows/update-dev-container-version.yml@d2a4595b1bfa59f04c7cecb080a126321d42a333
Copy link

Copilot AI Mar 25, 2026

Choose a reason for hiding this comment

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

The PR description/title focuses on Dependabot cooldown, but this PR also introduces a new scheduled workflow for updating the devcontainer version. Please either update the PR description to include this scope, or split the workflow addition into a separate PR so the change is easier to review and audit.

Copilot uses AI. Check for mistakes.
Comment on lines +12 to +14
contents: read
packages: read
pull-requests: write
Copy link

Copilot AI Mar 25, 2026

Choose a reason for hiding this comment

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

The permissions keys are indented more deeply than the rest of the job fields, which is inconsistent with other workflows in this repo (e.g. .github/workflows/pull_request.yml). Please align the indentation to the standard 2-space steps under permissions: to keep YAML formatting consistent and avoid future merge/lint issues.

Suggested change
contents: read
packages: read
pull-requests: write
contents: read
packages: read
pull-requests: write

Copilot uses AI. Check for mistakes.
@sonarqubecloud
Copy link
Copy Markdown

@anthony-nhs anthony-nhs merged commit d763348 into main Mar 27, 2026
63 of 66 checks passed
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