Merge pull request #24 from idleberg/exclude-node-modules

remove node_modules

.github/workflows/node.js.yml

@@ -36,10 +36,13 @@ jobs:
       run: sudo chown -R $(whoami) /usr/share/nsis/Plugins/
       if: ${{ matrix.os == 'ubuntu-20.04' }}
 
+    - name: 'Install npm dependencies'
+      run: npm ci
+
     - name: 'Test'
       run: |
-        npm test -- --reporter json --reporter-options output=./report-${{ matrix.os }}.json
-        cat ./report-${{ matrix.os }}.json | jq
+        npm test -- --reporter json --reporter-options output=${{ runner.temp }}/report-${{ matrix.os }}.json
+        cat ${{ runner.temp }}/report-${{ matrix.os }}.json | jq
 
     - name: 'Publish Tests'
       uses: 'dorny/test-reporter@v1'
@@ -49,5 +52,5 @@ jobs:
       continue-on-error: true
       with:
         name: 'Tests ${{ matrix.os }}'
-        path: 'report-${{ matrix.os }}.json'
+        path: ${{ runner.temp }}/report-${{ matrix.os }}.json
         reporter: mocha-json

.gitignore

@@ -5,3 +5,4 @@ nsis/nsis/
 .DS_Store
 .ionide
 report*.json
+node_modules/

README.md

@@ -39,6 +39,10 @@ Newline-delimited list of paths to load plugins from. Default `""`.
     arguments: "/V3"
 ```
 
+## Development
+
+This action needs to be built from the source code located in the `/src` folder. Whenever you make changes, you should run the `npm run build` script. Otherwise, your changes won't be tested by CI.
+
 ## Sample Projects
 
 * [joncloud/makensis-action-test](https://github.com/joncloud/makensis-action-test)

action.yml

@@ -18,7 +18,7 @@ inputs:
 
 runs:
   using: 'node16'
-  main: 'index.js'
+  main: 'dist/index.js'
 
 branding:
   icon: 'triangle'