-
Notifications
You must be signed in to change notification settings - Fork 502
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
Check files under .github updated #2780
base: main
Are you sure you want to change the base?
Conversation
Codecov ReportBase: 80.58% // Head: 80.58% // No change to project coverage 👍
Additional details and impacted files@@ Coverage Diff @@
## main #2780 +/- ##
=======================================
Coverage 80.58% 80.58%
=======================================
Files 149 149
Lines 2849 2849
Branches 198 198
=======================================
Hits 2296 2296
Misses 553 553 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
In the last commit I used the same artifactIds as in the play-json PR and also added the file where Scala Steward did actually update the version. It is failing now as reported in #2608. |
I also tried this case with #2816 but it is also failing there. |
I converted the test to a The following tests passes: test("Files under .github yml should be updated".only) {
val artifactIds = Nel.of("scala3-compiler".a, "scala3-library".a, "scala3-library_sjs1".a)
val update = ("org.scala-lang".g % artifactIds % "3.1.3-RC2" %> "3.1.3-RC3").group
val original = Map(
"project/Dependencies.scala" -> """val Scala3 = "3.1.3-RC2" """,
".github/workflows/ci.yml" -> """scala3: 2.12.15, 2.13.8, 3.1.3-RC2"""
)
val expected = Map(
"project/Dependencies.scala" -> """val Scala3 = "3.1.3-RC3" """,
".github/workflows/ci.yml" -> """scala3: 2.12.15, 2.13.8, 3.1.3-RC3"""
)
runApplyUpdate(update, original, expected)
} |
I think we can make it work, if we add |
Closes #2608