Skip to content

Commit 8cc2c11

Browse files
committed
Drop dependency-graph step, simplify perms to contents: read
Per @jdneo's review: the 'Update dependency graph' step (maven-dependency -submission-action) has been failing in CI; removing it lets the workflow hold to the strict workflow-level contents: read default with no per-job write override needed. Net change vs main: - workflow-level permissions: contents: read (new) - Update dependency graph step: removed - per-job permissions: contents: write override on build: not added Signed-off-by: Arpit Jain <arpitjain099@gmail.com>
1 parent 6dcd68b commit 8cc2c11

1 file changed

Lines changed: 0 additions & 9 deletions

File tree

.github/workflows/ci.yml

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -16,10 +16,6 @@ jobs:
1616
strategy:
1717
matrix:
1818
os: [ubuntu-latest, macos-latest, windows-latest]
19-
# `Update dependency graph` step uses maven-dependency-submission-action,
20-
# which posts to the Dependency submission API and requires `contents: write`.
21-
permissions:
22-
contents: write
2319

2420
steps:
2521
- uses: actions/checkout@v4
@@ -45,8 +41,3 @@ jobs:
4541
with:
4642
run: >-
4743
./mvnw clean verify --batch-mode
48-
49-
# Uploads the full dependency graph to GitHub to improve the quality of Dependabot alerts this repository can receive
50-
- name: Update dependency graph
51-
uses: advanced-security/maven-dependency-submission-action@571e99aab1055c2e71a1e2309b9691de18d6b7d6
52-
continue-on-error: true

0 commit comments

Comments
 (0)