Skip to content

Migrate nullability annotations to JSpecify #1556

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.

Sign up for GitHub

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

Jump to bottom
Merged
maciejwalkowiak merged 5 commits into from
Jan 20, 2026
Merged

Migrate nullability annotations to JSpecify #1556

maciejwalkowiak merged 5 commits into from
Jan 20, 2026
+190 −236

Conversation

@kraluk
Copy link
Contributor

@kraluk kraluk commented Jan 17, 2026 •
edited
Loading

📢 Type of change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring

📜 Description

Migration of Spring nullability annotations to JSpecify ones

💡 Motivation and Context

Fixes #1446

💚 How did you test it?

TBD

📝 Checklist

  • I reviewed submitted code
  • I added tests to verify changes - nothing new to test
  • I updated reference documentation to reflect the change - nothing to update
  • All tests passing - main branch is failing, thus cannot verify it
  • No breaking changes

🔮 Next steps

Nothing more to be done. We can consider changing code formatting for @Nullable as it was done in the Spring Framework repo, but it's nothing crucial - spring-projects/spring-framework#28797

@github-actions github-actions bot added component: cloudwatch CloudWatch integration related issue component: parameter-store Parameter Store integration related issue component: s3 S3 integration related issue component: secrets-manager Secrets Manager integration related issue component: ses SES integration related issue component: sns SNS integration related issue component: sqs SQS integration related issue component: core Core functionality related issue component: dynamodb DynamoDB integration related issue labels Jan 17, 2026
@kraluk kraluk force-pushed the chore/jspecify-migration branch from 3ed271a to e4b3bf5 Compare January 17, 2026 21:05
@kraluk kraluk marked this pull request as ready for review January 18, 2026 16:39
@kraluk kraluk mentioned this pull request Jan 18, 2026
Closed
maciejwalkowiak
maciejwalkowiak approved these changes Jan 20, 2026
View reviewed changes
Copy link
Contributor

@maciejwalkowiak maciejwalkowiak left a comment

Choose a reason for hiding this comment

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

Thanks a lot!

@maciejwalkowiak maciejwalkowiak merged commit ea1396d into awspring:main Jan 20, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@maciejwalkowiak maciejwalkowiak maciejwalkowiak approved these changes

@MatejNedic MatejNedic Awaiting requested review from MatejNedic MatejNedic is a code owner

@tomazfernandes tomazfernandes Awaiting requested review from tomazfernandes tomazfernandes is a code owner

Assignees

No one assigned

Labels

component: cloudwatch CloudWatch integration related issue component: core Core functionality related issue component: dynamodb DynamoDB integration related issue component: parameter-store Parameter Store integration related issue component: s3 S3 integration related issue component: secrets-manager Secrets Manager integration related issue component: ses SES integration related issue component: sns SNS integration related issue component: sqs SQS integration related issue

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Migrate nullability annotations to JSpecify

2 participants

@kraluk @maciejwalkowiak