Merge pull request #77 from MatteoPologruto/switch-to-ncc-packaging

Use ncc to compile the action

Author: MatteoPologruto

.github/CONTRIBUTING.md

@@ -41,29 +41,10 @@ npm run test
 
 ### 6. Build
 
-It is necessary to compile the code before it can be used by GitHub Actions. We check in the `node_modules` to provide runtime dependencies to the system using the Action, so be careful not to `git add` all the development dependencies you might have under your local `node_modules`.
-Remember to run these commands before committing any code changes:
+It is necessary to compile the code before it can be used by GitHub Actions. Remember to run these commands before committing any code changes:
 
 ```
-npm run build
-```
-
-remove all the dependencies:
-
-```
-rm -rf node_modules
-```
-
-add back **only** the runtime dependencies:
-
-```
-npm install --production
-```
-
-check in the code that matters:
-
-```
-git add lib node_modules
+task build
 ```
 
 ### 7. Commit

.github/workflows/check-packaging-ncc-typescript-npm.yml renamed to .github/workflows/check-packaging-ncc-typescript-task.yml

@@ -39,14 +39,16 @@ jobs:
         with:
           node-version: ${{ env.NODE_VERSION }}
 
-      - name: Install dependencies
-        run: npm install
+      - name: Install Task
+      uses: arduino/setup-task@v1
+      with:
+        repo-token: ${{ secrets.GITHUB_TOKEN }}
+        version: 3.x
 
       - name: Build project
-        run: |
-          npm run build
+        run: task ts:build
 
       - name: Check packaging
         # Ignoring CR because ncc's output has a mixture of line endings, while the repository should only contain
         # Unix-style EOL.
-        run: git diff --ignore-cr-at-eol --color --exit-code lib
+        run: git diff --ignore-cr-at-eol --color --exit-code dist