Skip to content

Commit

Permalink
defualts added to workflow, not action
Browse files Browse the repository at this point in the history
  • Loading branch information
zMynxx committed Aug 18, 2023
1 parent e13c4b3 commit 10539b4
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
7 changes: 2 additions & 5 deletions .github/actions/nodejs/npm-test/action.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -24,11 +24,6 @@ outputs:
description: 'My output description'
value: ${{ steps.test.outputs.test_result }}

defaults:
run:
shell: bash
working-directory: ${{ inputs.package_json_path }}

runs:
using: "composite"
steps:
Expand All @@ -47,10 +42,12 @@ runs:

- name: Install Dependencies
if: ${{ steps.cache-npm.outputs.cache-hit != 'true' }}
shell: bash
run: npm install ${{ inputs.force_install_dependencies && '--force' || '' }}

- name: Run tests
id: test
shell: bash
run: |
${{ inputs.test_command }}
echo "test_result=${{ job.status }}" >> $GITHUB_OUTPUT
3 changes: 3 additions & 0 deletions .github/workflows/ci-nodejs.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,9 @@ jobs:
strategy:
matrix:
version: [16, 17, 18]
defaults:
run:
shell: bash
steps:
- name: Checkout code (including tags)
uses: actions/checkout@v3
Expand Down

0 comments on commit 10539b4

Please sign in to comment.