Skip to content

Commit dac7bd1

Browse files
ci: Remove DangerJS from CI workflow
Remove the danger job that enforced manual changelog entries. This enables the transition to auto-generated changelogs. We considered modifying the action to only disable changelog enforcement while retaining other checks (action pinning, docs reminders), but decided these checks are not essential for the sentry-cli workflow. Fixes #3073 Co-Authored-By: Claude <[email protected]>
1 parent 7570215 commit dac7bd1

File tree

1 file changed

+2
-15
lines changed

1 file changed

+2
-15
lines changed

.github/workflows/ci.yml

Lines changed: 2 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -37,31 +37,18 @@ jobs:
3737
name: Test Swift
3838
uses: ./.github/workflows/swift-test.yml
3939

40-
danger:
41-
name: Run Danger Checks
42-
runs-on: ubuntu-24.04
43-
permissions:
44-
contents: read
45-
pull-requests: write
46-
statuses: write
47-
if: ${{ github.event_name == 'pull_request' }}
48-
steps:
49-
- name: Run Danger
50-
uses: getsentry/github-workflows/danger@13be9bec4ec5cd67061b747972b996e9c80f4f3b # 3.1.0
51-
5240
required:
5341
name: Check required jobs
5442
runs-on: ubuntu-24.04
55-
needs: [lint, test, test_node, test_swift, danger]
43+
needs: [lint, test, test_node, test_swift]
5644
if: always()
5745
steps:
5846
- name: Check for failure
5947
if: ${{
6048
needs.lint.result != 'success' ||
6149
needs.test.result != 'success' ||
6250
needs.test_node.result != 'success' ||
63-
needs.test_swift.result != 'success' ||
64-
(needs.danger.result != 'success' && needs.danger.result != 'skipped')
51+
needs.test_swift.result != 'success'
6552
}}
6653
run: |
6754
echo "One or more jobs failed"

0 commit comments

Comments
 (0)