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

Check files under .github updated #2780

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

exoego
Copy link
Contributor

@exoego exoego commented Nov 18, 2022

Closes #2608

@codecov
Copy link

codecov bot commented Nov 18, 2022

Codecov Report

Base: 80.58% // Head: 80.58% // No change to project coverage 👍

Coverage data is based on head (b9d4718) compared to base (b9d4718).
Patch has no changes to coverable lines.

❗ Current head b9d4718 differs from pull request most recent head 9d11154. Consider uploading reports for the commit 9d11154 to get more accurate results

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.
📢 Do you have feedback about the report comment? Let us know in this issue.

@fthomas
Copy link
Member

fthomas commented Dec 17, 2022

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.

@fthomas
Copy link
Member

fthomas commented Dec 17, 2022

I also tried this case with #2816 but it is also failing there.

@mzuehlke
Copy link
Member

mzuehlke commented Jan 6, 2023

I converted the test to a RewriteTest and found that the .github/ folder is not the problem, but the key scala in the ci.yml prevents the update. If I change the key to scala3it works.

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)
  }	

@fthomas
Copy link
Member

fthomas commented Jan 6, 2023

[...] but the key scala in the ci.yml prevents the update. If I change the key to scala3it works.

I think we can make it work, if we add scala as a search term if one the search terms is scala3. Or we strip trailing digits from all search terms.

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.

GitHub actions workflow .yml file not updated
3 participants