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 sbt-scalajs, scalajs-compiler, ... to 1.10.0 in series/0.23 #6245
Update sbt-scalajs, scalajs-compiler, ... to 1.10.0 in series/0.23 #6245
Conversation
302f49a
to
6d96ad9
Compare
If the build passes, absolutely merge. It's forward-compatible with 1.8.0 and 1.9.0 and furthermore fixes CVE-2022-28355 aka GHSA-j2f9-w8wh-9ww4. Otherwise, I think we should get this fixed via CE, see: Edit: seems the build is broken. So let's wait for CE 3.3.11. |
@Mergifyio update |
✅ Branch has been successfully updated |
@deprecated("Preserved for bincompat", "0.23.12") | ||
def apply[G[_], F[_]](headerName: CIString, http: Http[G, F], G: Sync[G]): Http[G, F] = | ||
apply(headerName)(http)(G, UUIDGen.fromSync(G)) | ||
|
||
def apply[G[_], F[_]]( | ||
headerName: CIString | ||
)(http: Http[G, F])(implicit G: Sync[G]): Http[G, F] = | ||
)(http: Http[G, F])(implicit G: Sync[G], gen: UUIDGen[G]): Http[G, F] = | ||
Kleisli[G, Request[F], Response[F]] { req => | ||
for { | ||
header <- req.headers.get(headerName).map(_.head) match { | ||
case None => G.delay(Header.Raw(headerName, UUID.randomUUID().toString())) | ||
case None => UUIDGen.randomString[G].map(Header.Raw(headerName, _)) |
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.
What's good for JS is good for JVM too.
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.
But, someone else should review my API changes :)
def apply[G[_], F[_]]( | ||
headerName: CIString | ||
)(http: Http[G, F])(implicit G: Sync[G]): Http[G, F] = | ||
)(http: Http[G, F])(implicit G: Sync[G], gen: UUIDGen[G]): Http[G, F] = |
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.
UUIDGen
is a typeclass? 🤯 Java enters the typeclass-room 😎
I think this is quite good, so let's go onward. |
Updates
from 1.9.0 to 1.10.0.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a
.scala-steward.conf
file.Have a fantastic day writing Scala!
Files still referring to the old version number
The following files still refer to the old version number (1.9.0).
You might want to review and update them manually.
Ignore future updates
Add this to your
.scala-steward.conf
file to ignore future updates of this dependency:labels: library-update, early-semver-minor, semver-spec-minor, old-version-remains, commit-count:1