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 scalameta to 4.8.8 #1846

Closed
wants to merge 1 commit into from
Closed

Conversation

scala-center-steward[bot]
Copy link
Contributor

About this PR

πŸ“¦ Updates org.scalameta:scalameta from 4.8.4 to 4.8.8

πŸ“œ GitHub Release Notes - Version Diff

Usage

βœ… Please merge!

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!

βš™ Adjust future updates

Add this to your .scala-steward.conf file to ignore future updates of this dependency:

updates.ignore = [ { groupId = "org.scalameta", artifactId = "scalameta" } ]

Or, add this to slow down future updates of this dependency:

dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.scalameta", artifactId = "scalameta" }
}]
labels: library-update, early-semver-patch, semver-spec-patch, version-scheme:semver-spec, commit-count:1

@bjaglin
Copy link
Collaborator

bjaglin commented Aug 23, 2023

I want to implement in sbt-scalafix what was suggested to be done upstream in sbt in #1825 before merging this as a patch release.

We could try to improve the error message in sbt when scalac-semanticdb is not found since it's currently not actionable at all:

[error] (update) sbt.librarymanagement.ResolveException: Error downloading org.scalameta:semanticdb-scalac_2.13.11:4.5.13
[error]   Not found
[error]   Not found
[error]   not found: /home/brice/.ivy2/local/org.scalameta/semanticdb-scalac_2.13.11/4.5.13/ivys/ivy.xml
[error]   not found: https://repo1.maven.org/maven2/org/scalameta/semanticdb-scalac_2.13.11/4.5.13/semanticdb-scalac_2.13.11-4.5.13.pom
[error] Total time: 1 s, completed Jul 19, 2023, 3:49:41β€―PM

I was thinking of recovering that exception by redefining update to add error handling (like described in https://www.scala-sbt.org/1.x/docs/Tasks.html#Handling+Failure).

@scala-center-steward
Copy link
Contributor Author

Superseded by #1852.

@tgodzik
Copy link
Contributor

tgodzik commented Sep 22, 2023

What do you think about the approach in scalameta/metals#5671 ? We generate the map of supported versions from sbt and it's 30 lines of code. Doesn't seem to take up a lot of time to run also

@bjaglin
Copy link
Collaborator

bjaglin commented Nov 10, 2023

Thanks for the link and sorry for the late follow-up! It does look very similar to what we already have on-demand in sbt-scalafix via scalafixEnable scalacenter/sbt-scalafix#292.

I'd rather stick to the status quo (i.e. running the semanticdb-scalac version lookup only via a specific command) for the following reasons:

  • doing IOs at sbt boot (settings initialization) is a source of non-deterministic behavior (sbt-scalafix is used on dev workstations and CI, so subject to proxies, etc)
  • scalacenter/sbt-scalafix@4ced96a does a decent job at explaining users what's going on if they try to run with an unpublished semanticdb-scalac version (and I believe we could try to push this upstream to improve sbt's SemanticdbPlugin if we do anything on that topic)
  • we no longer have this problem with Scala 3 as the semanticdb output is part of the compiler
  • the frequency of 2.x releases is low and the announcement/process on Scala Contributors is very good, so it's not a big deal to synchronize sbt-scalafix releases (which can detect/fix changes in the compiler like in 2.13.12)

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