Skip to content

ci: Remove Danger workflow and add changelog policy#405

Merged
buenaflor merged 3 commits into
mainfrom
ci/remove-danger-workflow
May 28, 2026
Merged

ci: Remove Danger workflow and add changelog policy#405
buenaflor merged 3 commits into
mainfrom
ci/remove-danger-workflow

Conversation

@buenaflor
Copy link
Copy Markdown
Contributor

Remove the Danger workflow from pull request CI.

The repository already uses Craft's automatic changelog policy and changelog preview workflow, so the extra Danger workflow is no longer needed for changelog enforcement.

Made with Cursor

Remove the Danger workflow because changelog handling is now covered by
Craft's automatic changelog policy and preview workflow.

Co-Authored-By: GPT-5.5 <noreply@openai.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Copilot AI review requested due to automatic review settings May 28, 2026 11:26
@buenaflor buenaflor requested a review from denrase as a code owner May 28, 2026 11:26
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

Removes the dedicated Danger GitHub Actions workflow used in pull request CI, relying instead on the repository’s existing Craft changelog enforcement/preview setup.

Changes:

  • Deleted .github/workflows/danger.yml, removing the getsentry/github-workflows/danger@v3 job from PR CI triggers.

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

@buenaflor buenaflor changed the title ci: Remove Danger workflow ci: Remove Danger workflow and add changelog policy May 28, 2026
buenaflor and others added 2 commits May 28, 2026 13:44
Use Craft's changelog preview workflow on pull requests and add release
category rules so changelog validation no longer relies on Danger.

Co-authored-by: Cursor <cursoragent@cursor.com>
Document branch naming and automatic changelog behavior so contributors
know how pull request metadata affects release notes.

Co-authored-by: Cursor <cursoragent@cursor.com>
Comment thread .github/workflows/changelog-preview.yml
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 58d343b. Configure here.

Comment thread .github/workflows/changelog-preview.yml
@buenaflor buenaflor merged commit 56d0ecd into main May 28, 2026
23 checks passed
@buenaflor buenaflor deleted the ci/remove-danger-workflow branch May 28, 2026 11:50
buenaflor added a commit that referenced this pull request May 28, 2026
* ci: Remove Danger workflow

Remove the Danger workflow because changelog handling is now covered by
Craft's automatic changelog policy and preview workflow.

Co-Authored-By: GPT-5.5 <noreply@openai.com>
Co-authored-by: Cursor <cursoragent@cursor.com>

* ci: Configure changelog preview

Use Craft's changelog preview workflow on pull requests and add release
category rules so changelog validation no longer relies on Danger.

Co-authored-by: Cursor <cursoragent@cursor.com>

* docs: Update contributing guide

Document branch naming and automatic changelog behavior so contributors
know how pull request metadata affects release notes.

Co-authored-by: Cursor <cursoragent@cursor.com>

---------

Co-authored-by: GPT-5.5 <noreply@openai.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
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.

3 participants