Repository navigation
Shorten purity warning suppression keys - #298
Conversation
The Checker Framework renames the message keys purity.not.deterministic.X, purity.not.sideeffectfree.X, and purity.not.deterministic.not.sideeffectfree.X to purity.X.
|
Understand this PR’s impact Explore downstream dependencies and potential security impact with Blast Radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository: typetools/jdk/.coderabbit.yaml Review profile: ASSERTIVE Plan: Advanced Run ID: 📒 Files selected for processing (5)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughUpdated five Suggested reviewers: Priority: ⬇️ Low Change: Refactor 🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Merge together:
typetools/checker-framework#8242
#298
plume-lib/bcel-util#584
plume-lib/javac-parse#312
plume-lib/merging#397
plume-lib/plume-util#786
plume-lib/reflection-util#545
The Checker Framework renames the message keys
purity.not.deterministic.X,purity.not.sideeffectfree.X, andpurity.not.deterministic.not.sideeffectfree.Xtopurity.X. This updates the warning suppressions accordingly.This does not compile until the corresponding Checker Framework change is released.