Skip to content

Commit

Permalink
Turn off automerge of updates
Browse files Browse the repository at this point in the history
Before replacing dependabot with a custom action, we were manually approving
PRs. I don't believe the change to a custom action was intended to change that
behaviour. There are additional security concerns that we should consider
before moving to automerge any updates.

Given that we review dependabot PRs once a week, I've also changed the workflow
to run weekly rather than daily (as dependabot was doing previously).
  • Loading branch information
lucyb committed Nov 4, 2024
1 parent 56b6ed4 commit eab036d
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion .github/workflows/update-dependencies.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ name: Update python dependencies
on:
workflow_dispatch:
schedule:
- cron: "5 6 * * *"
- cron: "5 6 * * MON"

jobs:
update-dependencies:
Expand All @@ -24,3 +24,4 @@ jobs:
- uses: opensafely-core/update-dependencies-action@v1
with:
token: ${{ steps.generate-token.outputs.token }}
automerge: false

0 comments on commit eab036d

Please sign in to comment.