-
Notifications
You must be signed in to change notification settings - Fork 0
Scala 3 🎉 #713
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
Scala 3 🎉 #713
Conversation
not sure if i love the lazy props since that makes importing them a bit weird.
4cf48bb to
d8298e4
Compare
d8298e4 to
06ee29c
Compare
gunnarvelle
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Kva var det som gjorde kompileringa så treig, som du snakka om?
|
|
||
| object SharedDependencies { | ||
| val ScalaVersion = "3.7.2" | ||
| val ScalaVersion = "3.3.6" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Koffor ikkje 3.7.2 som du hadde først?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Egentlig bare fordi jeg prøvde alt for å fikse den kompilatorbug'en i starten og 3.3.6 er LTS.
Men jeg kan egentlig godt oppgradere til 3.7.2 igjen
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Neida, gir meining med en LTS
audio-api/src/main/scala/no/ndla/audioapi/controller/SeriesController.scala
Outdated
Show resolved
Hide resolved
audio-api/src/main/scala/no/ndla/audioapi/controller/SeriesController.scala
Outdated
Show resolved
Hide resolved
article-api/src/main/scala/no/ndla/articleapi/controller/ArticleControllerV2.scala
Show resolved
Hide resolved
| import io.circe.generic.semiauto.{deriveDecoder, deriveEncoder} | ||
| import io.circe.{Decoder, Encoder} | ||
|
|
||
| // format: off |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💯
Rekkefølgen på Hvorfor det er sånn? Aner ikke, men det skiller ca ~450 sekunder på å kompilere |
Det er jo heilt vannvittig! |
Japp, det var veldig merkelig. Det gjør at det frister å legge til tester på kompileringstid på github actions 😂 |
gunnarvelle
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Har gjort noen stikkprøver på bygging og kjøring av apper lokalt, og alt ser ut til å funke fint. LGTM!
Denne blir naturligvis litt stor, men jeg tror å delvis migrere kodebasen kommer til å være mye mer hodebry så here goes.
De fleste endringene:
val Failure(ex) = sometrysyntax trigger en warning@uncheckedi tester for det.lazymed en som erlazyså nå er det consistent på tvers avComponentRegistryTestEnvironmentog i cakepattern traitene.propssom vi gjorde et par steder tidligere.ManifestmedClassTagSchemaog circe ting der vi tidligere brukte helauto.in(...)kallene pågetArticlesByIdsendepunktet til før.out, men jeg aner egentlig ikke hvorfor det er så veldig mye tyngere for kompilatoren å forstå i den gamle rekkefølgen.