-
Notifications
You must be signed in to change notification settings - Fork 202
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
Fix compile warnings in mantis-runtime module #396
Comments
@liuml07 I would like to work on this issue. |
Hi @liuml07 can you please tell me how to check for compile warnings in the code? Should I build the project using |
Building the top project will show compiler warnings for all modules. You can also check code warnings in IntelliJ IDE editor (open a |
@liuml07 in mantis-runtime we have different modules like command, computation etc. Is it fine if I change and raise pr module by module. |
Yeah, feel free to break down problems |
Made changes to module mantis-runtime/command resolving warnings like adding final keyword, converting lambda expression to method reference, and removing unnecessary initialization in PR #404 |
Hi @liuml07, is this issue still open and was it left by someone else? I'm a junior in college who interested in Netflix Summer 2024 SWE internship and Netflix open source, can I take over and continue working on this issue please? |
There are compile warnings in the code due to:
@Nullable/@NotNull
annotationsMap<String> m = new HashMap<String>()
)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:
This tracks the effort for the
mantis-runtime
module to make the code change scoped. Note this module contains more code than other modules.The text was updated successfully, but these errors were encountered: