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.5 #1825

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.2 to 4.8.5

πŸ“œ 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 Jul 19, 2023

Putting this on hold a bit.

The biggest impact of this comes from scalameta/scalameta#3250, which is effectively breaking, as it can cause regressions for people upgrading sbt-scalafix in their build since install instructions advise to use the scalameta version used in scalafix to fetch scalac-semanticdb. We might want to revisit that to put emphasis on using scalafixEnable which does not suffer from the regression thanks to scalacenter/sbt-scalafix#292 (and adds less friction for newcomers anyway).

Discussed on Scalameta's discord:

For users integrating scalafix in their build, the recommended setup is to use the scalameta version referenced by scalafix in sbt's SemanticdbPlugin, as the default baked-in in sbt tends to lag), so I expect some limited impact. As a first step, 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 in SemanticdbPlugin to add error handling (like described in https://www.scala-sbt.org/1.x/docs/Tasks.html#Handling+Failure). Thoughts?

@tgodzik
Copy link
Contributor

tgodzik commented Jul 19, 2023

Actually, it would be awesome to fetch the latest supported version to be built into sbt when using semanticdbEnabled, though I don't want to delve into sbt currently. I plan to update things in the metals sbt plugin for Metals and we could port the same thing to sbt-scalafix, what do you think?

@bjaglin
Copy link
Collaborator

bjaglin commented Jul 19, 2023

I plan to update things in the metals sbt plugin for Metals and we could port the same thing to sbt-scalafix, what do you think?

Actually, the logic is already there, but I am concerned about the IO overhead at sbt startup if we were to make that the default, as compiler plugins can only be added through setting keys from what I know.

@tgodzik
Copy link
Contributor

tgodzik commented Jul 19, 2023

I plan to update things in the metals sbt plugin for Metals and we could port the same thing to sbt-scalafix, what do you think?

Actually, the logic is already there, but I am concerned about the IO overhead at sbt startup if we were to make that the default, as compiler plugins can only be added through setting keys from what I know.

I wonder if we could cache that information, maybe that would help? πŸ€”

But anyway, thanks for sharing that, I might need to use the same mechanism.

@scala-center-steward
Copy link
Contributor Author

Superseded by #1835.

@scala-center-steward scala-center-steward bot deleted the update/scalameta-4.8.5 branch July 27, 2023 00:21
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