Skip to content

Conversation

vaind
Copy link
Contributor

@vaind vaind commented Oct 10, 2025

Summary

Migrates the Danger workflow from v2 (reusable workflow) to v3 (composite action).

Benefits

  • Latest Danger JS version (v13.0.4)
  • Better conventional commit scope handling
  • Enhanced support for non-conventional PR titles

#skip-changelog

🤖 Generated with Claude Code

Converts danger workflow from reusable workflow (v2) to composite
action (v3). This is a breaking change that requires restructuring
the job definition.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
cursor[bot]

This comment was marked as outdated.

@lcian lcian enabled auto-merge (squash) October 10, 2025 14:58
@vaind
Copy link
Contributor Author

vaind commented Oct 14, 2025

@lcian I don't have permissions to merge in this repo

@lcian
Copy link
Member

lcian commented Oct 14, 2025

@vaind don't worry about it, as you might have seen a workflow was failing due to a failure in the Sentry Prevent action and that workflow is required.
It appears the problem hasn't been fixed yet - it has to do with running from a fork.
When it's fixed, I'll merge this 👍

@vaind
Copy link
Contributor Author

vaind commented Oct 14, 2025

feel free to just push to the repo directly, IDC about the credit for the change.

@lcian lcian merged commit ca23268 into getsentry:master Oct 14, 2025
15 of 18 checks passed
@vaind vaind deleted the chore/migrate-danger-v3 branch October 14, 2025 14:21
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.

2 participants