Rewrite action

* replace docker action with composite github action
* update references
* upgrade actions/checkout to v4

* switch from scalafmt:v2.3.2

* support specifying an arbitrary version of scalafmt
  (as long as it's available as `scalafmt-linux-glibc` in releases)

* take responsibility for this action
* drop GitHub Marketplace instructions (this will come back once we
  publish)
* drop references to HCL syntax (GitHub Actions Alpha is sufficiently
  obsolete that no one remembers it)

.github/regenerate-good.sh

@@ -0,0 +1,14 @@
+#!/bin/sh
+act -W .github/workflows/test-regenerate-good.yml 2>/dev/null |
+perl -ne '
+    unless ($state) {
+        $state = 1 if (/run-scalafmt\.sh/);
+        next;
+    }
+    if ($state == 1) {
+        $state = 2 if (/docker exec/);
+        next;
+    }
+    last unless s/^.*\| //;
+    print;
+' > .github/testdata/Good.scala

.github/testdata/.scalafmt.conf

@@ -1,3 +1,5 @@
-# I dont like this formatting directive in reality, but lets
+# I don't like this formatting directive in reality, but it lets
 # us verify that this is reading a conf file easily
-align = most
+align.preset = most
+version = 3.7.15
+runner.dialect = scala213

.github/testdata/Bad.scala

@@ -16,7 +16,6 @@ sealed abstract class Formatted {
 }
 
 /** Aligned by first asterisk, default ScalaDoc style is second.
- *
  */
 object Formatted {
     case class Success(formattedCode: String) extends Formatted

.github/testdata/Good.scala

@@ -3,6 +3,7 @@ messiness added to it. -mroth */
 package org.scalafmt
 
 sealed abstract class Formatted {
+
   def toEither: Either[Throwable, String] = this match {
     case Formatted.Success(s) => Right(s)
     case Formatted.Failure(e) => Left(e) // too much space before comment
@@ -15,7 +16,6 @@ sealed abstract class Formatted {
 }
 
 /** Aligned by first asterisk, default ScalaDoc style is second.
-  *
   */
 object Formatted {
   case class Success(formattedCode: String) extends Formatted

.github/workflows/test-all.yml

@@ -0,0 +1,12 @@
+on: [workflow_dispatch]
+name: Check with all testdata
+
+jobs:
+  check-testdata:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - name: Checking testdata
+        uses: ./
+        with:
+          args: "-c .github/testdata/.scalafmt.conf --list"

.github/workflows/test-regenerate-good.yml

@@ -0,0 +1,12 @@
+on: [workflow_dispatch]
+name: Regenerate Good file (run in act)
+
+jobs:
+  regenerate-good:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - name: Checking testdata
+        uses: ./
+        with:
+          args: "-c .github/testdata/.scalafmt.conf --stdout .github/testdata/Bad.scala"

.github/workflows/test.yml

@@ -1,12 +1,12 @@
 on: [push]
-name: Check with testdata
+name: Check with good testdata
 
 jobs:
-  check-testdata:
+  check-good-testdata:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v1
-      - name: Checking testdata
+      - uses: actions/checkout@v4
+      - name: Checking testdata (pass)
         uses: ./
         with:
           args: "-c .github/testdata/.scalafmt.conf --exclude .github/testdata/Bad.scala --list"

Makefile

@@ -1,34 +1,17 @@
-IMAGE := openlaw/scalafmt-ci
-# DEFAULT_ARGS := --list
-WORKDIR := /github/workspace
 TESTDATA := .github/testdata
 
 .PHONY: test testfail testdata clean
 
-image:
-	docker build -t $(IMAGE) .
-
 test:
-	docker run --rm -it \
-		-w=$(WORKDIR) \
-		-v "$$(pwd)/$(TESTDATA)":$(WORKDIR) \
-		$(IMAGE) --list --exclude Bad.scala
+	act -W .github/workflows/test.yml
 
 # expected failure, to see output
 testfail: 
-	docker run --rm -it \
-		-w=$(WORKDIR) \
-		-v "$$(pwd)/$(TESTDATA)":$(WORKDIR) \
-		$(IMAGE) --list
+	act -W .github/workflows/test-all.yml
 
 # generate the "good" version of the sample file
 $(TESTDATA)/Good.scala: $(TESTDATA)/Bad.scala $(TESTDATA)/.scalafmt.conf
-	docker run --rm -i \
-		-v "$$(pwd)/$(TESTDATA)":/conf \
-		mrothy/scalafmt-native:2.2.2 \
-			--config /conf/.scalafmt.conf \
-			--stdin --stdout \
-	< $< > $@
+	./.github/regenerate-good.sh
 
 testdata: $(TESTDATA)/Good.scala
 

README.md

@@ -11,7 +11,7 @@ Simply add a step such as the following to your your workflow yml file:
 
 ```yml
 - name: Check for scalafmt conformance
-  uses: openlawteam/scalafmt-ci@v2
+  uses: garnercorp/scalafmt-ci@v3
 ```
 
 Example in the full context of a workflow file, with some optional arguments:
@@ -24,28 +24,11 @@ jobs:
   scalafmt-lint:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v1
+      - uses: actions/checkout@v4
         with:
           fetch-depth: 1
       - name: Checking your code to see if u r naughty or nice
-        uses: openlawteam/scalafmt-ci@v2
+        uses: garnercorp/scalafmt-ci@v3
         with:
           args: "--exclude=third_party --list"
 ```
-
-### Installing with GitHub Marketplace
-
-https://github.com/marketplace/actions/scalafmt-action
-
-### (Legacy) HCL Syntax
-
-If you are still using legacy HCL format (from the GitHub Actions Alpha test),
-you will need to pin to a previous version of this action.
-
-```hcl
-action "lint" {
-  uses="openlawteam/scalafmt-ci@v0"
-  # optional additional args
-  args="--exclude ./vendor"
-}
-```

action.yml

@@ -1,6 +1,6 @@
 name: "Scalafmt Action"
 description: "Format Scala code using scalafmt"
-author: "Matthew Rothenberg <[email protected]>"
+author: "GarnerCorp <[email protected]>"
 branding:
   icon: "wind"
   color: "red"
@@ -9,8 +9,31 @@ inputs:
     description: "Args to pass to scalafmt"
     default: "--list"
     required: false
+  version:
+    description: "Version of scalafmt"
+    default: "3.7.15"
+    required: false
+
 runs:
-  using: "docker"
-  image: "Dockerfile"
-  args:
-    - ${{ inputs.args }}
+  using: "composite"
+  steps:
+  - shell: bash
+    run: |
+      ${{ github.action_path }}/set-path.sh
+    env:
+      scalafmt_version: ${{ inputs.version }}
+  - uses: actions/cache@v3
+    id: cache
+    with:
+      path: ${{ env.SCALAFMT_PATH }}
+      key: scalafmt-${{ inputs.version }}
+  - shell: bash
+    if: ${{ !steps.cache.outputs.cache-hit }}
+    run: |
+      curl -sL https://github.com/scalameta/scalafmt/releases/download/v${scalafmt_version}/scalafmt-linux-glibc > "$SCALAFMT_PATH/scalafmt"
+      chmod +x "$SCALAFMT_PATH/scalafmt"
+  - shell: bash
+    run: |
+      ${{ github.action_path }}/run-scalafmt.sh
+    env:
+      args: ${{ inputs.args }}

run-scalafmt.sh

@@ -0,0 +1,2 @@
+#!/bin/sh
+scalafmt --non-interactive $args

set-path.sh

@@ -0,0 +1,8 @@
+#!/bin/sh
+SCALAFMT_PATH="/tmp/scalafmt/${scalafmt_version}"
+mkdir -p "$SCALAFMT_PATH"
+(
+    echo "SCALAFMT_PATH=$SCALAFMT_PATH"
+    echo "scalafmt_version=$scalafmt_version"
+) >> "$GITHUB_ENV"
+echo "$SCALAFMT_PATH" >> "$GITHUB_PATH"