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

[2.9.x] Pin Play 2.9 updates #520

Merged
merged 1 commit into from
Jan 15, 2024

Conversation

ihostage
Copy link
Member

No description provided.

@ihostage ihostage requested a review from mkurz January 15, 2024 12:50
@ihostage ihostage force-pushed the steward-message-29x branch from b105a1b to 852ea24 Compare January 15, 2024 13:11
@mkurz mkurz merged commit ce2eb46 into playframework:2.9.x Jan 15, 2024
12 checks passed

updates.pin = [
// Play 2.9
{ groupId = "com.typesafe.play", artifactId = "sbt-plugin", version = "2.9" },
Copy link
Member

Choose a reason for hiding this comment

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

Ops, too fast, seems the version need to have a . as suffix: https://github.com/scala-steward-org/scala-steward/blob/main/docs/faq.md#how-can-version-updates-be-controlled

So here it should be 2.9. not just 2.9...

Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Member Author

Choose a reason for hiding this comment

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

Hmm 🤔 Will this config not work without ending .? Or it's just possible to get PR with bump to 2.90.0? 😄

Copy link
Member

Choose a reason for hiding this comment

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

To be honest I did not check if it is working or not, I just saw that in the docs and also in our other repos we always add . at the end, so I think if we know this is working we probably keep it that way. Maybe it also works without the dot... I don't know.

Copy link
Member Author

Choose a reason for hiding this comment

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

Ok. Sorry for that.

@ihostage ihostage deleted the steward-message-29x branch January 16, 2024 11:48
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