Skip to content
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

Ignore the warningAsError (-Werror) maven build flag #2841

Closed

Conversation

micrommer
Copy link

Update Scala to the latest version and ignore the maven strict warning checking flag!
With these changes, the project has been built with no problem using the following command
mvn clean install

It is a showcase pull request and more investigation is required.
A draft for #2811.

@micrommer
Copy link
Author

BTW, I don't have any problem with compiling the project using the following JDK

================================== 🦐 Java version 🦐 ==================================
openjdk version "1.8.0_402"
OpenJDK Runtime Environment (Zulu 8.76.0.17-CA-macos-aarch64) (build 1.8.0_402-b06)
OpenJDK 64-Bit Server VM (Zulu 8.76.0.17-CA-macos-aarch64) (build 25.402-b06, mixed mode)
================================== 🦐 Maven version 🦐 ==================================
Apache Maven 3.9.6 (bc0240f3c744dd6b6ec2920b3cd08dcc295161ae)
Maven home: /opt/homebrew/opt/sdkman-cli/libexec/candidates/maven/current
Java version: 1.8.0_402, vendor: Azul Systems, Inc., runtime: /Users/dbk/Library/Java/JavaVirtualMachines/azul-1.8.0_402/Contents/Home/jre
Default locale: en_IR, platform encoding: UTF-8
OS name: "mac os x", version: "15.0", arch: "aarch64", family: "mac"

@pivovarit pivovarit added this to the v0.10.5 milestone Sep 29, 2024
@pivovarit
Copy link
Member

Thanks for the PR - it's clear the only thing left is to deal with those serialization issues :) someone is already on it

@pivovarit pivovarit closed this Sep 29, 2024
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