Skip to content

Update to NullAway 0.13.6, and add missing nullness annotation to address new warning#36850

Open
msridhar wants to merge 1 commit into
spring-projects:mainfrom
msridhar:fix-warnings-upcoming
Open

Update to NullAway 0.13.6, and add missing nullness annotation to address new warning#36850
msridhar wants to merge 1 commit into
spring-projects:mainfrom
msridhar:fix-warnings-upcoming

Conversation

@msridhar
Copy link
Copy Markdown
Contributor

@msridhar msridhar commented May 27, 2026

Updates to NullAway 0.13.6. This version produces a new warning, which is fixed by adding a @Nullable annotation. From the surrounding code it is clear that valueIterator should have type Iterator<@Nullable Object>.

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged or decided on label May 27, 2026
@msridhar msridhar marked this pull request as draft May 27, 2026 17:13
@msridhar
Copy link
Copy Markdown
Contributor Author

Sorry, with this fix the code doesn't pass the currently-released version of NullAway. Converted to draft; will mark ready when the new version of NullAway is released.

org.springframework.web.util.UriComponents.VarArgsTemplateVariables to fix
new warning.

Signed-off-by: Manu Sridharan <msridhar@gmail.com>
@msridhar msridhar force-pushed the fix-warnings-upcoming branch from 31ccf45 to d9e1f94 Compare June 5, 2026 17:36
@msridhar msridhar changed the title Add missing nullness annotation in org.springframework.web.util.UriComponents.VarArgsTemplateVariables Update to NullAway 0.13.6, and add missing nullness annotation to address new warning Jun 5, 2026
@msridhar msridhar marked this pull request as ready for review June 5, 2026 17:37
@msridhar
Copy link
Copy Markdown
Contributor Author

msridhar commented Jun 5, 2026

This PR is now up to date with the latest NullAway release (0.13.6) and could be landed. Possibly it's desirable to update the NullAway version in the Spring Nullability plugin, cut a release, and then update the plugin version here, but hopefully this PR is still useful.

@sdeleuze
Copy link
Copy Markdown
Contributor

sdeleuze commented Jun 5, 2026

Thanks @msridhar!

@sdeleuze sdeleuze added type: task A general task and removed status: waiting-for-triage An issue we've not yet triaged or decided on labels Jun 5, 2026
@sdeleuze sdeleuze self-assigned this Jun 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: task A general task

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants