Modernize scala syntax - Githubissues

cloudant-labs / clouseau

Expose Lucene features as an erlang-like node

Apache License 2.0

58 stars 32 forks source link

Modernize scala syntax #87

Closed iilyak closed 7 months ago

iilyak commented 7 months ago

This PR removes deprecated forms of Scala syntax as well as

try to use Option type whenever possible
validates the types of inner elements in nested structures
add test cases for parsing

The main goal of the PR is to prepare the codebase for Scala version upgrade.

pgj commented 7 months ago

Are you planning to update the CI as well? Apparently it tries to build the code with old Java (and Scala) and it fails.

iilyak commented 7 months ago

Are you planning to update the CI as well? Apparently it tries to build the code with old Java (and Scala) and it fails.

This work doesn't make it possible to build full project with new Java/Scala. I just renovated syntax of *.scala files without addressing the dependencies.

Therefore updating CI is not needed.