-
Notifications
You must be signed in to change notification settings - Fork 47
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
Ensure SBT is installed using Coursier #669
Conversation
a140810
to
89338eb
Compare
89338eb
to
6648ba3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think then the GHA setp:
- name: Install sbt
uses: sbt/setup-sbt@v1
can be removed in the README.md (multiple times)
6648ba3
to
4bec440
Compare
4bec440
to
2d41fa7
Compare
A snapshot release has been created as You can test it out with: uses: scala-steward-org/scala-steward-action@snapshots/669 It will be automatically recreated on any change to this PR. |
Thanks @mzuehlke! Just made the change to README. I have tested it on runners with/without SBT pre-installed and it works fine in both of them. |
@@ -10,7 +10,7 @@ import {type NonEmptyString} from '../core/types' | |||
* Installs `coursier` and add its executable to the `PATH`. | |||
* | |||
* Once coursier is installed, installs `scalafmt` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
* Once coursier is installed, installs `scalafmt` | |
* Once coursier is installed, installs `scalafmt`, |
Fixes #663