Update CI and test

nwhetsell · Aug 21, 2024 · 03eb69c · 03eb69c
1 parent d6cdb9a
commit 03eb69c
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 13 deletions.
diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml
@@ -2,9 +2,6 @@ name: CI
 
 on: [push, pull_request]
 
-env:
-  CI: true
-
 jobs:
   Test:
     strategy:
@@ -15,21 +12,21 @@ jobs:
     - name: Check out repository
       uses: actions/checkout@v4
 
-    - name: Install Pulsar on macOS
-      run: brew install pulsar
-      if: ${{ matrix.os == 'macos-latest' }}
     - name: Install Pulsar on Linux
       uses: pulsar-edit/action-pulsar-dependency@v3
       if: ${{ matrix.os == 'ubuntu-latest' }}
+    - name: Install Pulsar on macOS
+      run: brew install pulsar
+      if: ${{ matrix.os == 'macos-latest' }}
 
     - name: Install LilyPond on Linux
       run: |
-        wget --no-verbose https://gitlab.com/lilypond/lilypond/-/releases/v2.24.3/downloads/lilypond-2.24.3-linux-x86_64.tar.gz
-        tar -xf lilypond-2.24.3-linux-x86_64.tar.gz
-        echo "$(pwd)/lilypond-2.24.3/bin" >> $GITHUB_PATH
+        wget --no-verbose https://gitlab.com/lilypond/lilypond/-/releases/v2.24.4/downloads/lilypond-2.24.4-linux-x86_64.tar.gz
+        tar -xf lilypond-2.24.4-linux-x86_64.tar.gz
+        echo "$(pwd)/lilypond-2.24.4/bin" >> $GITHUB_PATH
       if: ${{ matrix.os == 'ubuntu-latest' }}
     - name: Install LilyPond on macOS
-      run: brew update && brew install lilypond
+      run: brew install lilypond
       if: ${{ matrix.os == 'macos-latest' }}
 
     - name: Install package dependencies

diff --git a/spec/linter-lilypond-spec.js b/spec/linter-lilypond-spec.js
@@ -1,18 +1,20 @@
 const dedent = require("dedent-js");
 const fs = require("fs");
 const path = require("path");
+const process = require("process");
 
 const lint = require("../lib/linter-lilypond").provideLinter().lint;
 
 describe("linter-lilypond", () => {
-  const preamble = '\\version "2.22.0"\n';
+  const preamble = '\\version "2.24.4"\n';
 
   beforeEach(() => {
     waitsForPromise(() => atom.packages.activatePackage("linter-lilypond"));
   });
 
   // These tests pass locally, but not on GitHub Actions.
-  /*
+  if (process.env.CI !== "true") {
+
   describe("LilyPond linter", () => {
     it("lints a valid file", () => {
       const filePath = path.join(__dirname, "test.ly");
@@ -125,7 +127,8 @@ describe("linter-lilypond", () => {
       }));
     });
   });
-  */
+
+  }
 
   describe("LilyPond grammar", () => {
     let grammar;