Skip to content

ci(breaking-change-detector): don't use maintain-one-comment and instead do it manually#22568

Merged
rluvaton merged 7 commits into
apache:mainfrom
rluvaton:check-breaking-change-detector
May 28, 2026
Merged

ci(breaking-change-detector): don't use maintain-one-comment and instead do it manually#22568
rluvaton merged 7 commits into
apache:mainfrom
rluvaton:check-breaking-change-detector

Conversation

@rluvaton

@rluvaton rluvaton commented May 27, 2026

Copy link
Copy Markdown
Member

the comment and label for breaking change detector is not working for 8 days already, (it was noticed by @neilconway)

after searching the issue is that actions-cool/maintain-one-comment that we used was compromised and was removed from github.

we did not used the malicious commit fortunately.

this change the comment logic to be manual instead of using that action

See more:

Issue to remove the action from asf allow list:

the comment for breaking change detector looks broken since last success
was 8 days ago
@github-actions github-actions Bot added the core Core DataFusion crate label May 27, 2026
@github-actions github-actions Bot added common Related to common crate and removed core Core DataFusion crate labels May 27, 2026
@github-actions github-actions Bot added development-process Related to development process of DataFusion and removed common Related to common crate labels May 27, 2026
@rluvaton rluvaton closed this May 27, 2026
@rluvaton rluvaton reopened this May 27, 2026
@rluvaton rluvaton changed the title ci-test: trigger breaking change detector ci(breaking-change-detector): don't use maintain-one-comment and instead do it manually May 27, 2026
@rluvaton rluvaton marked this pull request as ready for review May 27, 2026 20:31
@rluvaton

Copy link
Copy Markdown
Member Author

@Jefffrey can you please review this real quick?

@rluvaton

Copy link
Copy Markdown
Member Author

@alamb maybe can you review?

@alamb alamb left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Sure -- let's try it. Thanks for trying to keep things moving. It is hard to keep all this stuff maintained

@rluvaton rluvaton added this pull request to the merge queue May 28, 2026
Merged via the queue into apache:main with commit f70dacb May 28, 2026
35 checks passed
@rluvaton rluvaton deleted the check-breaking-change-detector branch May 28, 2026 16:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

development-process Related to development process of DataFusion

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants