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

Backport 4.5.8 to scala 2.12 #185

Open
wants to merge 4 commits into
base: 4.x
Choose a base branch
from
Open

Backport 4.5.8 to scala 2.12 #185

wants to merge 4 commits into from

Conversation

b-gyula
Copy link

@b-gyula b-gyula commented Aug 25, 2024

I don't know why scala 2 was abadoned, but I made a backport.
I can maintain if you like. In a separate branch?

  • merged form 992b89e
  • made scala version configurable through properties in pom.xmls

- merge form 5c3006f
- make scala version configurable through properties in `pom.xml`s
- Remove generated sources
- Merge 234addd adding itests
- `tupleOf` helper function to use instead of  `Tuple.of` that gives `ambiguous reference to overloaded definition` error
- `execute` extension method with varargs not requiring Tuple for `PreparedQuery.execute`
- Add scalacOption: -Xsource:3
@oliver-brm
Copy link
Collaborator

Hi there and thanks for the PR! ♥

We moved to Scala 3 because we wanted to keep it simple and maintain only one Scala code generator. Plus, Scala 3 has some nice features which we may use eventually and which could be hard to backport.

Feel free to fork this repo if you would like to maintain a Scala 2 branch of vertx-lang-scala.

@b-gyula
Copy link
Author

b-gyula commented Aug 30, 2024

Do you create a branch for it in this repo or I should publish it with a different org id?

@oliver-brm
Copy link
Collaborator

oliver-brm commented Sep 2, 2024

Do you create a branch for it in this repo or I should publish it with a different org id?

Not sure what you mean with org id. You can create a fork under your own account. In that case, I would set a different Maven groupId. Does that answer your question?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants