-
Notifications
You must be signed in to change notification settings - Fork 209
Open
Labels
enhancementNew feature or requestNew feature or requestgood first issueGood for newcomersGood for newcomers
Description
There are compile warnings in the code due to:
- deprecated APIs
- missing
@Nullable/@NotNull
annotations - incomplete javadocs
- lambda can be replaced with method references
- raw usage of generic types
- unnecessary type parameters (like
Map<String> m = new HashMap<String>()
) - anonymous class can be replaced with lambdas
- ... and more.
We don't need to solve all of them, but a good coding exercise is to address most those compile warnings when possible. Guidelines are:
- Not change code logic, which should be addressed and filed PR separately
- Not change public interfaces (class, methods and fields) that would break other modules or applications. But it's safe to rename private fields or methods when necessary.
- Prefer readability to more concise code
- Improve tests in the module is welcome but not required
- Compile and pass all existing tests.
This tracks the effort for the mantis-runtime
module to make the code change scoped. Note this module contains more code than other modules.
Metadata
Metadata
Assignees
Labels
enhancementNew feature or requestNew feature or requestgood first issueGood for newcomersGood for newcomers