Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create a bot to check if there is a CHANGES entry for new PRs #13898

Open
stefanvodita opened this issue Oct 12, 2024 · 10 comments · Fixed by #14279
Open

Create a bot to check if there is a CHANGES entry for new PRs #13898

stefanvodita opened this issue Oct 12, 2024 · 10 comments · Fixed by #14279

Comments

@stefanvodita
Copy link
Contributor

Description

We always have to remind ourselves and others to add a CHANGES entry. We can probably write a GitHub action to check for changes to that file on every new PR and leave a comment warning the author if that's missing.

@prudhvigodithi
Copy link
Contributor

prudhvigodithi commented Oct 12, 2024

Here is the sample PR with change-log verifier action added prudhvigodithi#1, we can add the same workflow to lucene repo. But I see the error dangoslen/changelog-enforcer@v3 is not allowed to be used in apache/lucene.

@javanna
Copy link
Contributor

javanna commented Oct 14, 2024

Yes to this! It would be great to combine this with setting the milestone :)

@stefanvodita
Copy link
Contributor Author

@prudhvigodithi - I opened #13898 inspired by your PR, tested it on my fork, and it's working correctly.
@javanna - I like that idea! Maybe we can do that as a follow-up.

@stefanvodita
Copy link
Contributor Author

stefanvodita commented Mar 2, 2025

I was looking at the workflow that @pseudo-nymous wrote. The logs for #14276 say there is no CHANGES entry. We'll need to fix that. I've also added the PR number to the output (#14324), since I couldn't see it anywhere.

Edit: Same issue in the logs for #14320.
Edit: We even have runs that error out.

@stefanvodita stefanvodita reopened this Mar 2, 2025
@pseudo-nymous
Copy link
Contributor

Thanks for updating the script with PR number. I'll root cause and fix the issue.

@stefanvodita
Copy link
Contributor Author

@pseudo-nymous, I've attempted a fix in #14355.

@pseudo-nymous
Copy link
Contributor

Thanks @stefanvodita! I will take a look at the fix and propose changes accordingly.

@dweiss
Copy link
Contributor

dweiss commented Mar 15, 2025

dangoslen/changelog-enforcer@v3 is not allowed to be used in apache/lucene

Apache projects are not allowed to use arbitrary actions for security reasons, please take a look at:
https://infra.apache.org/github-actions-policy.html#External%5C%20actions

@stefanvodita
Copy link
Contributor Author

Just to clarify - the restriction @dweiss mentioned applies to the changelog-enforcer action, but not to the checkout action we are using.

@pseudo-nymous - I'm seeing new failures now, e.g. a fatal.
It's frustrating that the tests we perform on our forks aren't reflective of the behaviour we see in the main repo.

@pseudo-nymous
Copy link
Contributor

@stefanvodita I have added a fix for it. Please take a look. #14369

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants