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 ammonite to Scala 3 toolchain #1289

Merged
merged 2 commits into from
May 9, 2024

Conversation

lolgab
Copy link
Contributor

@lolgab lolgab commented Nov 16, 2023

@alexarchambault alexarchambault marked this pull request as ready for review February 23, 2024 14:18
Seems some of its classes are a problem for the next commit's changes,
giving errors like:
```
Error:  error while loading SocketType,
Error:  class file org/zeromq/SocketType.class is broken, reading aborted with class java.lang.RuntimeException
Error:  bad constant pool index: 0 at pos: 1840
Error:  -- [E006] Not Found Error: /home/runner/work/almond/almond/modules/shared/channels/src/main/scala/almond/channels/zeromq/ZeromqSocketImpl.scala:20:14
Error:  20 |  socketType: SocketType,
Error:     |              ^^^^^^^^^^
Error:     |              Not found: type SocketType
Error:     |
Error:     | longer explanation available when compiling with `-explain`
```

This reverts commit e6dc050.
@alexarchambault alexarchambault force-pushed the update-ammonite branch 4 times, most recently from a7b6264 to 2013165 Compare May 9, 2024 08:53
Co-Authored-By: Alexandre Archambault <alexandre.archambault@gmail.com>
@alexarchambault alexarchambault merged commit 40556c8 into almond-sh:main May 9, 2024
18 checks passed
@alexarchambault alexarchambault changed the title [WIP] Update ammonite to Scala 3 toolchain Update ammonite to Scala 3 toolchain May 9, 2024
@alexarchambault
Copy link
Member

Merged, thanks @lolgab! (for this PR, and the corresponding work in Ammonite too)

@lolgab lolgab deleted the update-ammonite branch November 1, 2024 14:08
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