Skip to content

Conversation

uschindler
Copy link
Contributor

This closes #15246.

I'd like to backport this to 10.x branch, but its not urgent.

Copy link
Contributor

github-actions bot commented Oct 4, 2025

This PR does not have an entry in lucene/CHANGES.txt. Consider adding one. If the PR doesn't need a changelog entry, then add the skip-changelog label to it and you will stop receiving this reminder on future updates to the PR.

@uschindler uschindler added the skip-changelog Apply to PRs that don't need a changelog entry, stopping the automated changelog check. label Oct 4, 2025
@uschindler
Copy link
Contributor Author

I think this is ready, no review required @dweiss !

@uschindler uschindler merged commit ecd681a into apache:main Oct 4, 2025
12 checks passed
@uschindler
Copy link
Contributor Author

I forgot to remove the import... done in 6b2268c

Why did code checks not notice that?

@uschindler uschindler deleted the dev/forbiddenapis-3.10 branch October 4, 2025 23:35
Copy link
Contributor

@dweiss dweiss left a comment

Choose a reason for hiding this comment

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

LGTM, thanks Uwe.

@dweiss
Copy link
Contributor

dweiss commented Oct 5, 2025

Why did code checks not notice that?

This import was in a .gradle file and none of the linters touch that at the moment, I think.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

legacy-jira-label:forbiddenapis module:build-infra skip-changelog Apply to PRs that don't need a changelog entry, stopping the automated changelog check.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Re-enable forbidden-apis checker when it supports JDK 25

2 participants