Skip to content

Commit 47be28c

Browse files
fix: tune Dependabot config and fix DCO assistant bugs (#534)
* fix: restrict Dependabot pip updates to security-only The Dependabot config added in #517 included weekly version-bump PRs for all three pip packages. This would generate noisy PRs for routine dep updates we don't need. Set open-pull-requests-limit: 0 on the pip ecosystems so only CVE-triggered security updates open PRs. GitHub Actions weekly bumps are kept as-is to keep SHA pins current. * fix: group Dependabot Actions PRs and fix DCO allowlist - Add a Dependabot group to bundle all GitHub Actions updates into a single weekly PR instead of one per action - Fix DCO allowlist: dependabot -> dependabot[bot] to match the actual GitHub username (the old value never matched, but there were no Dependabot PRs before #517 to expose the bug) * fix: align DCO assistant if-condition with custom sign-off text The step's if-condition checked for the default sign-off text but custom-pr-sign-comment uses different wording. This meant the issue_comment trigger was always skipped - sign-offs only worked by accident when a subsequent push re-triggered the action via pull_request_target.
1 parent 2528741 commit 47be28c

File tree

2 files changed

+9
-2
lines changed

2 files changed

+9
-2
lines changed

.github/dependabot.yml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,23 +4,30 @@ updates:
44
directory: /
55
schedule:
66
interval: weekly
7+
groups:
8+
all-actions:
9+
patterns:
10+
- "*"
711
commit-message:
812
prefix: "ci"
913
- package-ecosystem: pip
1014
directory: /packages/data-designer-config
1115
schedule:
1216
interval: weekly
17+
open-pull-requests-limit: 0
1318
commit-message:
1419
prefix: "chore"
1520
- package-ecosystem: pip
1621
directory: /packages/data-designer-engine
1722
schedule:
1823
interval: weekly
24+
open-pull-requests-limit: 0
1925
commit-message:
2026
prefix: "chore"
2127
- package-ecosystem: pip
2228
directory: /packages/data-designer
2329
schedule:
2430
interval: weekly
31+
open-pull-requests-limit: 0
2532
commit-message:
2633
prefix: "chore"

.github/workflows/dco-assistant.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ jobs:
2626
runs-on: ubuntu-latest
2727
steps:
2828
- name: "DCO Assistant"
29-
if: (github.event.comment.body == 'recheck' || github.event.comment.body == 'I have read the Contributor Agreement including DCO and I hereby sign the Contributor Agreement and DCO') || github.event_name == 'pull_request_target'
29+
if: (github.event.comment.body == 'recheck' || github.event.comment.body == 'I have read the DCO document and I hereby sign the DCO.') || github.event_name == 'pull_request_target'
3030
uses: contributor-assistant/github-action@ca4a40a7d1004f18d9960b404b97e5f30a505a08
3131
env:
3232
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
@@ -35,7 +35,7 @@ jobs:
3535
path-to-signatures: "dco-signatures.json"
3636
path-to-document: 'https://github.com/NVIDIA-NeMo/DataDesigner/blob/main/DCO'
3737
branch: 'signatures'
38-
allowlist: dependabot
38+
allowlist: dependabot[bot]
3939
create-file-commit-message: "chore: create file to store dco signatures"
4040
signed-commit-message: "chore: $contributorName has signed the dco in #$pullRequestNo"
4141
custom-notsigned-prcomment: "Thank you for your submission! We ask that $you sign our [Developer Certificate of Origin](https://github.com/NVIDIA-NeMo/DataDesigner/blob/main/DCO) before we can accept your contribution. You can sign the DCO by adding a comment below using this text:"

0 commit comments

Comments
 (0)