Merge pull request #12 from technote-space/release/v1.0.4

Release/v1.0.4

Author: technote-space

.github/workflows/ci.yml

@@ -17,7 +17,6 @@ jobs:
         with:
           fetch-depth: 3
       - uses: technote-space/get-diff-action@gh-actions
-        id: git-diff
         with:
           PREFIX_FILTER: |
             src/
@@ -27,21 +26,21 @@ jobs:
       - name: Get Yarn Cache Directory
         id: yarn-cache
         run: echo "::set-output name=dir::$(yarn cache dir)"
-        if: steps.git-diff.outputs.diff
+        if: env.GIT_DIFF
       - name: Cache node dependencies
         uses: actions/cache@v1
         with:
           path: ${{ steps.yarn-cache.outputs.dir }}
           key: ${{ runner.os }}-yarn-${{ hashFiles('**/yarn.lock') }}
           restore-keys: |
             ${{ runner.os }}-yarn-
-        if: steps.git-diff.outputs.diff
+        if: env.GIT_DIFF
       - name: Install Package dependencies
         run: yarn install
-        if: steps.git-diff.outputs.diff
+        if: env.GIT_DIFF
       - name: Check code style
-        run: yarn eslint ${{ steps.git-diff.outputs.diff }}
-        if: steps.git-diff.outputs.diff
+        run: yarn eslint ${{ env.GIT_DIFF }}
+        if: env.GIT_DIFF
       - uses: 8398a7/action-slack@v1
         with:
           type: failure
@@ -62,7 +61,6 @@ jobs:
         with:
           fetch-depth: 3
       - uses: technote-space/get-diff-action@gh-actions
-        id: git-diff
         with:
           PREFIX_FILTER: |
             src/
@@ -75,31 +73,31 @@ jobs:
         uses: actions/setup-node@v1
         with:
           node-version: ${{ matrix.node }}
-        if: steps.git-diff.outputs.diff || github.event_name == 'push'
+        if: env.GIT_DIFF || github.event_name == 'push'
       - name: Get Yarn Cache Directory
         id: yarn-cache
         run: echo "::set-output name=dir::$(yarn cache dir)"
-        if: steps.git-diff.outputs.diff || github.event_name == 'push'
+        if: env.GIT_DIFF || github.event_name == 'push'
       - name: Cache node dependencies
         uses: actions/cache@v1
         with:
           path: ${{ steps.yarn-cache.outputs.dir }}
           key: ${{ runner.os }}-yarn-${{ hashFiles('**/yarn.lock') }}
           restore-keys: |
             ${{ runner.os }}-yarn-
-        if: steps.git-diff.outputs.diff || github.event_name == 'push'
+        if: env.GIT_DIFF || github.event_name == 'push'
       - name: Install Package dependencies
         run: yarn install
-        if: steps.git-diff.outputs.diff || github.event_name == 'push'
+        if: env.GIT_DIFF || github.event_name == 'push'
       - name: Run tests
         run: yarn cover
-        if: steps.git-diff.outputs.diff || github.event_name == 'push'
+        if: env.GIT_DIFF || github.event_name == 'push'
       - name: Codecov
         run: curl -s https://codecov.io/bash | bash -s -- -t $CODECOV_TOKEN -f $COVERAGE_FILE
         env:
           CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}
           COVERAGE_FILE: ./coverage/lcov.info
-        if: matrix.node == '12' && (steps.git-diff.outputs.diff || github.event_name == 'push')
+        if: matrix.node == '12' && (env.GIT_DIFF || github.event_name == 'push')
       - uses: 8398a7/action-slack@v1
         with:
           type: failure

README.md

@@ -41,19 +41,18 @@ jobs:
         with:
           fetch-depth: 3
       - uses: technote-space/get-diff-action@v1
-        id: git-diff
         with:
           PREFIX_FILTER: |
             src
             __tests__
           SUFFIX_FILTER: .ts
       - name: Install Package dependencies
         run: yarn install
-        if: steps.git-diff.outputs.diff
+        if: env.GIT_DIFF
       - name: Check code style
         # Check only the source codes that have differences
-        run: yarn eslint ${{ steps.git-diff.outputs.diff }}
-        if: steps.git-diff.outputs.diff
+        run: yarn eslint ${{ env.GIT_DIFF }}
+        if: env.GIT_DIFF
 ```
 
 If there is no difference in the source code below, this workflow will skip the code style check

__tests__/utils/command.test.ts

@@ -54,6 +54,7 @@ describe('getGitDiff', () => {
 		process.env.INPUT_PREFIX_FILTER = 'src/\n__tests__';
 		process.env.INPUT_SUFFIX_FILTER = '.ts\n.txt';
 		process.env.INPUT_ABSOLUTE      = 'false';
+		process.env.INPUT_SET_ENV_NAME  = '';
 		const mockExec                  = spyOnExec();
 		setChildProcessParams({
 			stdout: (command: string): string => {

__tests__/utils/misc.test.ts

@@ -64,18 +64,18 @@ describe('setResult', () => {
 
 		stdoutCalledWith(mockStdout, [
 			'::set-output name=diff::test1 test2',
+			'::set-env name=GIT_DIFF::test1 test2',
 		]);
 	});
 
-	it('should set result with env', () => {
-		process.env.INPUT_SET_ENV_NAME = 'DIFF';
+	it('should set result without env', () => {
+		process.env.INPUT_SET_ENV_NAME = '';
 		const mockStdout               = spyOnStdout();
 
 		setResult(['test1', 'test2']);
 
 		stdoutCalledWith(mockStdout, [
 			'::set-output name=diff::test1 test2',
-			'::set-env name=DIFF::test1 test2',
 		]);
 	});
 });

action.yml

@@ -38,6 +38,7 @@ inputs:
     required: false
   SET_ENV_NAME:
     description: Env name.
+    default: GIT_DIFF
     required: false
 outputs:
   diff:

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@technote-space/get-diff-action",
-  "version": "1.0.3",
+  "version": "1.0.4",
   "description": "GitHub actions to get git diff.",
   "author": "Technote <[email protected]> (https://technote.space)",
   "license": "MIT",