Skip to content

Update snakeyaml to 2.6#877

Open
typelevel-steward[bot] wants to merge 1 commit intomainfrom
update/snakeyaml-2.6
Open

Update snakeyaml to 2.6#877
typelevel-steward[bot] wants to merge 1 commit intomainfrom
update/snakeyaml-2.6

Conversation

@typelevel-steward
Copy link
Contributor

About this PR

📦 Updates org.yaml:snakeyaml from 1.33 to 2.6

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!

🔍 Files still referring to the old version number

The following files still refer to the old version number (1.33).
You might want to review and update them manually.

docs/plugins.svg
⚙ Adjust future updates

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

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

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

dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.yaml", artifactId = "snakeyaml" }
}]
labels: library-update, old-version-remains, commit-count:1

@mergify mergify bot requested a review from armanbilge February 27, 2026 00:18
@satorg
Copy link
Contributor

satorg commented Feb 27, 2026

I guess we should bump up the sbt-typelvel plugin version to 0.9.0 once we merge this PR, correct?

@armanbilge
Copy link
Member

I guess we should bump up the sbt-typelvel plugin version to 0.9.0 once we merge this PR, correct?

Actually, the tlBaseVersion should be bumped in the PR that makes the breaking change, not after.

Also, this breaking bump of snakeyaml must be paired with the corresponding bump in circe-yaml. #715

I think I avoided doing either of these updates for so long for fear of a classpath clash with some other sbt-plugin pinned to an old version of these YAML libraries 🤔

Copy link
Member

@armanbilge armanbilge left a comment

Choose a reason for hiding this comment

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

at the very least, must also bump circe and the base version

@satorg
Copy link
Contributor

satorg commented Feb 27, 2026

@armanbilge ,

Also, this breaking bump of snakeyaml must be paired with the corresponding bump in circe-yaml. #715

Agree – that's what I'm thinking as well.

Actually, the tlBaseVersion should be bumped in the PR that makes the breaking change, not after.

This PR does it too: #835. I guess, it would be nice to squeeze them both into the same release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants