-
Notifications
You must be signed in to change notification settings - Fork 720
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
Modify the implementation of isUnnamedClass() #17766
Conversation
See #17725 (comment) Both |
PR to re-enable excluded tests adoptium/aqa-tests#4664 |
Unfortunately my comment has been ignored, the aqa-tests PR is merged. |
Pls set it as a draft in the future. |
+1 - we have a few reviewers in that repo that do not honour those type of requests, so best to keep in draft until dependent issues resolved. |
Change the order of the checks to do modifier check first Closes eclipse-openj9#17725 Signed-off-by: Hang Shao <hangshao@ca.ibm.com>
jenkins test sanity alinux64 jdk21 |
Change the order of the checks to do modifier check first
Closes #17725