Skip to content

Use DataFlowIssue as suppression name alias for NullAway #4779

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

Merged
merged 2 commits into from
Aug 11, 2025

Conversation

scordio
Copy link
Contributor

@scordio scordio commented Jul 19, 2025

I am working on a NullAway enhancement that allows setting suppression name aliases, like the JetBrains DataFlowIssue inspection.

As I'm also testing my changes against the JUnit codebase, I'm opening this PR as a draft to showcase the idea, and I'd be happy to finalize it if the team agrees.

Prerequisites

  • NullAway 0.12.8 for the new SuppressionNameAliases config option
  • New release of gradle-nullaway-plugin for the corresponding DSL (workaround demonstrated in the PR)

I hereby agree to the terms of the JUnit Contributor License Agreement.


Definition of Done

@scordio scordio marked this pull request as draft July 19, 2025 14:27
@marcphilipp
Copy link
Member

Looks nice! Alternatively, it Would be nice if NullAway and IntelliJ could agree on a warning category or IntelliJ could recognize NullAway.

@scordio scordio changed the title Set DataFlowIssue as NullAway suppression name alias Bump NullAway to 0.12.8, set DataFlowIssue as suppression name alias Aug 8, 2025
@marcphilipp
Copy link
Member

@scordio Any objections on merging this without the new DSL?

@scordio
Copy link
Contributor Author

scordio commented Aug 11, 2025

No objections! I will be able to rebase this PR only later in the day.

I might contribute to the plugin project too but it'll take a few more days before I can jump on it.

@marcphilipp marcphilipp marked this pull request as ready for review August 11, 2025 09:34
@marcphilipp marcphilipp changed the title Bump NullAway to 0.12.8, set DataFlowIssue as suppression name alias Configure DataFlowIssue as suppression name alias Aug 11, 2025
@marcphilipp marcphilipp changed the title Configure DataFlowIssue as suppression name alias Configure and use DataFlowIssue as suppression name alias Aug 11, 2025
@marcphilipp marcphilipp changed the title Configure and use DataFlowIssue as suppression name alias Use DataFlowIssue as suppression name alias for NullAway Aug 11, 2025
@marcphilipp marcphilipp merged commit 7ebd0a7 into junit-team:main Aug 11, 2025
14 checks passed
@scordio scordio deleted the nullaway-0.12.8 branch August 11, 2025 11: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