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

Update to Scala 3.4.0 #76

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

davoclavo
Copy link
Contributor

@davoclavo davoclavo commented Jun 27, 2024

Hello!

Here are some changes to update to Scala 3.4.0.
Match types seem to be behaving slightly different now, I wonder what improvements could be adopted in the future.

Some changes:

  1. Due to multiple new warnings:
    • Added infix modifier for commonly used infix operators and types - the other option is to remove the infix modifier and instead surround the infix operand with backticks
    • Replaced : _* for *
  2. Due to new compilation errors:
    • Added Promotion (T, DType) => T - Although I'm still unsure what are the implications of this change everything runs properly in my machine
  3. Updated SBT + Plugins in order to fix github action workflow generation

@davoclavo davoclavo force-pushed the update_to_scala_3_4_0 branch 2 times, most recently from 94a1c59 to b95787f Compare June 28, 2024 04:49
Due to multiple new warnings:

- Added infix modifier for commonly used infix operators and types
- Replaced `: _*` for `*`

Due to new compilation errors:

- Added Promotion `(T, DType) => T` -- match types are behaving slightly
different now.
  - Although I'm still unsure what are the implications of this change
  everything runs properly
@davoclavo davoclavo force-pushed the update_to_scala_3_4_0 branch from b95787f to dc691f0 Compare June 28, 2024 04:53
@sbrunk
Copy link
Owner

sbrunk commented Jul 19, 2024

Hey thanks @davoclavo and sorry for missing this. I'm a bit hesitant to move to 3.4 already as I think it can't be used from someone on 3.3 LTS if I understand the compat rules correctly.

@davoclavo
Copy link
Contributor Author

Hey @sbrunk no worries! Happy to leave this PR in the back burner for now, until a solution for the problem you mentioned is found.

Is cross compilation to multiple versions something that might be worth looking into?

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