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

Adopt JSpecify #890

Merged
merged 18 commits into from
Jan 4, 2025
Merged

Adopt JSpecify #890

merged 18 commits into from
Jan 4, 2025

Conversation

jeffgbutler
Copy link
Member

In keeping with the direction of many other projects (including The Spring Framework), we've adopted JSpecify to fully document the null handling in this library.

There should be very few - or no - runtime changes based on the adoption of JSpecify, but there will potentially be IDE and tooling warnings generated by this change, Please read the release notes in the CHANGELOG for full details.

In addition, JSpecify is now a runtime dependency of the library.

@coveralls
Copy link

coveralls commented Jan 3, 2025

Coverage Status

coverage: 100.0%. remained the same
when pulling 4f81c07 on jeffgbutler:jspecify
into 73bb838 on mybatis:master.

@jeffgbutler jeffgbutler merged commit 697b1dc into mybatis:master Jan 4, 2025
7 checks passed
@jeffgbutler jeffgbutler deleted the jspecify branch January 4, 2025 12:46
@jeffgbutler jeffgbutler added this to the 2.0.0 milestone Jan 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants