Merge pull request #804 from wellcomecollection/test-autoformat-githu…

…b-workflow

run auto formating script with github action

.buildkite/pipeline.yml

@@ -1,9 +1,6 @@
 steps:
-  - command: |
-      ./builds/run_formatting.sh
-      .buildkite/scripts/push_autoformatting_changes.sh
-      ./builds/run_linting.sh
-    label: "autoformat"
+  - command: ./builds/run_linting.sh
+    label: "linting"
     agents:
       queue: "scala"
 

.github/workflows/autoformat.yml

@@ -0,0 +1,13 @@
+# Runs auto-formatting script on push to any branch
+name: "Scala auto-formatting"
+
+on: push
+
+permissions:
+  id-token: write
+  contents: write
+
+jobs:
+  autoformat:
+    uses: wellcomecollection/.github/.github/workflows/scala_formatting.yml@main
+    secrets: inherit

builds/run_sbt_task_in_docker.sh

@@ -48,6 +48,9 @@ else
 fi
 
 docker run --tty --rm \
+  -e AWS_ACCESS_KEY_ID="${AWS_ACCESS_KEY_ID:-}" \
+  -e AWS_SECRET_KEY="${AWS_SECRET_KEY:-}" \
+  -e AWS_SESSION_TOKEN="${AWS_SESSION_TOKEN:-}" \
   --volume ~/.sbt:/root/.sbt \
   --volume ~/.ivy2:/root/.ivy2 \
   --volume "$HOST_COURSIER_CACHE:/root/$LINUX_COURSIER_CACHE" \

items/src/main/scala/weco/api/items/services/SierraItemUpdater.scala

@@ -194,9 +194,8 @@ class SierraItemUpdater(
   def updateItems(items: Seq[DisplayItem]): Future[Seq[DisplayItem]] = {
     val staleItemIds = items
       .filter(item => item.isStale)
-      .map(
-        item => SierraItemIdentifier.fromSourceIdentifier(item.identifiers.head)
-      )
+      .map(item =>
+        SierraItemIdentifier.fromSourceIdentifier(item.identifiers.head))
 
     staleItemIds.size match {
       case 0 => Future.successful(items)
@@ -214,7 +213,5 @@ class SierraItemUpdater(
   private def parseISOStringToLocalDate(isoString: String): LocalDate =
     LocalDate.parse(
       isoString,
-      DateTimeFormatter.ofPattern("yyyy-MM-dd'T'HH:mm:ss.SSSXXX")
-    )
-
+      DateTimeFormatter.ofPattern("yyyy-MM-dd'T'HH:mm:ss.SSSXXX"))
 }