Skip to content

Use JSpecify static analysis annotations - #442

Open
joshuajaharwood wants to merge 8 commits into
lessthanoptimal:SNAPSHOTfrom
joshuajaharwood:feature-jspecify
Open

joshuajaharwood wants to merge 8 commits into
lessthanoptimal:SNAPSHOTfrom
joshuajaharwood:feature-jspecify

Conversation

@joshuajaharwood

Copy link
Copy Markdown
  • Bump spotless plugin to latest
  • Only use JSpecify annotations, remove Jetbrains and JSR-305 dependencies
  • Make all usages compliant with annotation method contracts

@lessthanoptimal

Copy link
Copy Markdown
Owner

Thanks for the PR. Could you resolve the merge conflicts. That is likely caused by the recent update to Java 25 and several libraries being updated. If this goes well I'll look into updating updating other libraries that I maintain.

Based on a conversation with GPT, it looks like we might need to set -XDaddTypeAnnotationsToSymbol=true to true because the byte code generated is for Java 11. Could you investigate that too?

@joshuajaharwood

Copy link
Copy Markdown
Author

I will get to this eventually, I'm just absolutely hammered with work right now!

@lessthanoptimal

Copy link
Copy Markdown
Owner

I can relate. No problem!

@lessthanoptimal
lessthanoptimal force-pushed the SNAPSHOT branch 2 times, most recently from 719d6b9 to 9d5eaa4 Compare June 17, 2026 01:09
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.

2 participants