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 coursier to 2.1.18 and coursier-interface to 1.0.24 #3950

Merged
merged 12 commits into from
Nov 22, 2024

Conversation

alexarchambault
Copy link
Collaborator

@alexarchambault alexarchambault commented Nov 12, 2024

coursier 2.1.18 takes Maven "BOMs" better into account, which can sometimes significantly change the versions (or even the dependencies) selected by coursier (see also the 2.1.17 release notes).

#3949 offers a way to disable the new way of processing BOMs, if ever that's needed, with

def resolutionParams = Task.Anon {
  super.resolutionParams().withEnableDependencyOverrides(Some(false))
}

coursier-interface is bumped along the way just in case. Its 1.0.24 version uses coursier 2.1.18 under-the-hood (in a hidden proguarded fashion).

The latter using the former under-the-hood (in a hidden proguarded fashion)
@alexarchambault

This comment was marked as resolved.

@alexarchambault alexarchambault changed the title Update coursier to 2.1.17 and coursier-interface to 1.0.23 Update coursier to 2.1.18 and coursier-interface to 1.0.24 Nov 19, 2024
@alexarchambault

This comment was marked as resolved.

@alexarchambault alexarchambault force-pushed the coursier-2.1.17 branch 2 times, most recently from 390809c to 544f319 Compare November 20, 2024 14:08
@alexarchambault alexarchambault marked this pull request as ready for review November 20, 2024 14:19
docs/package.mill Outdated Show resolved Hide resolved
lihaoyi pushed a commit that referenced this pull request Nov 21, 2024
Needed for #3950. It's running
into a [scaladoc bug](scala/bug#10028), and
would need to exclude an internal dependency of coursier from the class
path passed to scaladoc.

Once merged, Mill would need to be rebootstrapped on the latest nightly
version, and #3950 would
override `UnidocModule#unidocCompileClasspath` to exclude the JAR of
coursier/dependency from it.
build.mill Show resolved Hide resolved
Copy link
Member

@lefou lefou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. The unidoc workaround is a bit nasty but probably good enough for now.

@lihaoyi lihaoyi merged commit eb11ca9 into com-lihaoyi:main Nov 22, 2024
27 checks passed
@alexarchambault alexarchambault deleted the coursier-2.1.17 branch November 22, 2024 10:03
@lefou lefou added this to the 0.12.3 milestone Nov 23, 2024
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.

3 participants