-
Notifications
You must be signed in to change notification settings - Fork 78
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 deprecation warnings #85
Comments
Since the project is on gradle now exclusively, what command would reproduce this? |
Thanks good point. To see the warnings, add this to the bottom of allprojects {
gradle.projectsEvaluated {
tasks.withType(JavaCompile) {
options.compilerArgs << "-Xlint:deprecation"
}
}
} (specifically, as of now the warnings are:
) |
adding Not marking the methods as deprecated is one solution, but I think those annotations are potentially quite useful for catching bugs, so... 🤔 maybe best to just leave it be |
mvn clean compile -Dmaven.compiler.showDeprecation=true
…they don't indicate any actual problem, they're just because we have to implement mutators to fulfill Collections interfaces, but we have also deprecated mutators in PCollections so that people get IDE warnings if they try to use them
possibly can suppress the warning with a suppression comment?
The text was updated successfully, but these errors were encountered: