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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Only create scalafix job for 2.13 #6361

Merged
merged 1 commit into from
May 5, 2022

Conversation

armanbilge
Copy link
Member

First of all the extra jobs were worthless, because they weren't doing anything.

Secondly, because they ran so quickly (i.e. without starting sbt and downloading stuff) they were the first in line to upload the cache ... of nothing 馃檭

Fixes #6360.

@armanbilge armanbilge added the behind-the-scenes Appreciated, but not user-facing label May 2, 2022
@mergify mergify bot added the series/0.23 PRs targeting 0.23.x label May 2, 2022
@@ -39,10 +39,9 @@ ThisBuild / githubWorkflowAddedJobs ++= Seq(
WorkflowStep.Run(
List("cd scalafix", "sbt ci"),
Copy link
Member Author

Choose a reason for hiding this comment

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

Still, this fast-running job is not particularly helpful for our cache, since it doesn't use the primary build.sbt. This is something to fix in sbt-typelevel.

@danicheg danicheg merged commit fab4d97 into http4s:series/0.23 May 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
behind-the-scenes Appreciated, but not user-facing series/0.23 PRs targeting 0.23.x
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CI cache step isn't caching anything ???
2 participants