Skip to content

Conversation

@acm19
Copy link
Owner

@acm19 acm19 commented Dec 10, 2025

Issue #, if available:

Description of changes:

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@acm19 acm19 force-pushed the danger branch 2 times, most recently from df02513 to c3bef33 Compare December 10, 2025 13:59
@dblock
Copy link
Collaborator

dblock commented Dec 10, 2025

The token thing means that the token will get revoked quickly. The "right" approach is to use dblock/danger-changelog#65, then you don't need the token as the CI runs on PR, but the comment gets appended on pull request target.

@acm19 acm19 force-pushed the danger branch 2 times, most recently from 46899bd to c03a0a4 Compare December 10, 2025 15:05
Repository owner deleted a comment from github-actions bot Dec 10, 2025
@github-actions
Copy link

1 Error
🚫 Please put back the * Your contribution here. line into CHANGELOG.md.

Generated by 🚫 Danger

@acm19
Copy link
Owner Author

acm19 commented Dec 10, 2025

@dblock I made it work here, but the error is quite cryptic. It doesn't really explain why it's not happy with the changelog. Maybe some config I'm missing.

@dblock
Copy link
Collaborator

dblock commented Dec 10, 2025

@dblock I made it work here, but the error is quite cryptic. It doesn't really explain why it's not happy with the changelog. Maybe some config I'm missing.

It adds a comment to PRs, #155 (comment), and yes it should log it in the logs but I think it doesn't.

#155 (comment)

This works here because you're not working off a fork, don't merge it.

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