Skip to content

Conversation

@tomjenkinson
Copy link
Member

JDK17 JDK 17

CORE

@tomjenkinson
Copy link
Member Author

@jmfinelli @mmusgrov I think the failure can be ignored, but on the other hand I think this is now doing what we want so I will merge it when I get an approval from either of you. Thank you

Copy link
Member

@mmusgrov mmusgrov left a comment

Choose a reason for hiding this comment

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

Agreed, the fix worked.

@jmfinelli
Copy link
Contributor

parallel approval ;-)

@tomjenkinson tomjenkinson merged commit a2b2962 into main Jan 12, 2026
10 of 11 checks passed
@tomjenkinson
Copy link
Member Author

Thank you

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.

4 participants