Skip to content
This repository has been archived by the owner on Jul 14, 2023. It is now read-only.

GitHub actions fixes #190

Merged
merged 2 commits into from
Jul 2, 2021
Merged

GitHub actions fixes #190

merged 2 commits into from
Jul 2, 2021

Conversation

bjaglin
Copy link
Contributor

@bjaglin bjaglin commented Jul 1, 2021

No description provided.

@bjaglin bjaglin changed the title CI: fix publish workflow broken by 1589fa8 GitHub actions fixes Jul 1, 2021
@codecov
Copy link

codecov bot commented Jul 1, 2021

Codecov Report

Merging #190 (503a92e) into master (1589fa8) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #190   +/-   ##
=======================================
  Coverage   91.01%   91.01%           
=======================================
  Files           4        4           
  Lines         267      267           
  Branches       14       14           
=======================================
  Hits          243      243           
  Misses         24       24           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1589fa8...503a92e. Read the comment docs.

@bjaglin bjaglin marked this pull request as ready for review July 1, 2021 21:12
on:
push:
branches:
- main
Copy link
Owner

Choose a reason for hiding this comment

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

Hmm, we don't have a main branch?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

oops, bad copy/paste... amended!

@@ -14,7 +14,7 @@ jobs:
- uses: olafurpg/setup-scala@v11
- uses: coursier/cache-action@v6
- name: Test
run: sbt scalafmtCheck "rules/scalafix --check" +test
run: sbt scalafmtCheck "rules2_12/scalafix --check" +test
Copy link
Owner

Choose a reason for hiding this comment

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

Thanks for the fix!

Copy link
Owner

@liancheng liancheng left a comment

Choose a reason for hiding this comment

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

Thanks, merging!

@liancheng liancheng merged commit 10d7401 into liancheng:master Jul 2, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants