Skip to content

Commit

Permalink
Fix: Remove deprecated set-output command (#718)
Browse files Browse the repository at this point in the history
  • Loading branch information
damacus authored Jul 16, 2024
1 parent a4d8ae6 commit 4f384c4
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 15 deletions.
15 changes: 7 additions & 8 deletions action/diff/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ inputs:
required: false
python-version:
description: Python version used for running flux-local
default: '3.10'
default: "3.10"
strip-attrs:
description: Labels and annotations that should be stripped to reduce diff noise
default: helm.sh/chart,checksum/config,app.kubernetes.io/version,chart
Expand All @@ -39,13 +39,13 @@ inputs:
default: 10000
kustomize-build-flags:
description: Additional flags to pass to kustomize build
default: ''
default: ""
sources:
description: GitRepository or OCIRepository to include with optional source mappings like `flux-system` or `cluster=./kubernetes/`
default: ''
default: ""
token:
description: Token used for checkout action
default: '${{ github.token }}'
default: "${{ github.token }}"
required: false
outputs:
diff:
Expand All @@ -55,17 +55,16 @@ runs:
using: "composite"
steps:
- name: Verify Flux CLI
run:
flux --version || (echo "Could not find flux CLI, add https://fluxcd.io/flux/flux-gh-action/" && exit 1)
run: flux --version || (echo "Could not find flux CLI, add https://fluxcd.io/flux/flux-gh-action/" && exit 1)
shell: bash
- name: Copy requirements locally
id: copy-requirements
shell: bash
run: |
# `cache-dependency-path` seems to need to be within the current directory. Use a temporary directory
tempdir=$(mktemp --directory --tmpdir=. --suffix=-flux-local-diff-action)
cp ${{ github.action_path }}/../../requirements.txt $tempdir
echo "::set-output name=directory::${tempdir}"
cp ${{ github.action_path }}/../../requirements.txt ${tempdir}
echo "directory=${tempdir}" >> $GITHUB_OUTPUT
- name: Set up Python
uses: actions/setup-python@v5
with:
Expand Down
13 changes: 6 additions & 7 deletions action/test/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -22,22 +22,21 @@ inputs:
default: v1.9.0
python-version:
description: Python version used for running flux-local
default: '3.10'
default: "3.10"
debug:
description: When true, uses the DEBUG log level
default: false
kustomize-build-flags:
description: Additional flags to pass to kustomize build
default: ''
default: ""
sources:
description: GitRepository or OCIRepository to include with optional source mappings like `flux-system` or `cluster=./kubernetes/`
default: ''
default: ""
runs:
using: "composite"
steps:
- name: Verify Flux CLI
run:
flux --version || (echo "Could not find flux CLI, add https://fluxcd.io/flux/flux-gh-action/" && exit 1)
run: flux --version || (echo "Could not find flux CLI, add https://fluxcd.io/flux/flux-gh-action/" && exit 1)
shell: bash
- name: Copy requirements locally
id: copy-requirements
Expand All @@ -46,7 +45,7 @@ runs:
# `cache-dependency-path` seems to need to be within the current directory. Use a temporary directory
tempdir=$(mktemp --directory --tmpdir=. --suffix=-flux-local-diff-action)
cp ${{ github.action_path }}/../../requirements.txt $tempdir
echo "::set-output name=directory::${tempdir}"
echo "directory=${tempdir}" >> $GITHUB_OUTPUT
- name: Set up Python
uses: actions/setup-python@v4
with:
Expand All @@ -61,7 +60,7 @@ runs:
- uses: supplypike/setup-bin@v3
with:
uri: https://github.com/kyverno/kyverno/releases/download/${{ inputs.kyverno-version }}/kyverno-cli_${{ inputs.kyverno-version }}_linux_x86_64.tar.gz
name: 'kyverno-cli'
name: "kyverno-cli"
version: ${{ inputs.kyverno-version }}
if: ${{ inputs.enable-kyverno == 'true' }}
- name: Test cluster (flux-local)
Expand Down

0 comments on commit 4f384c4

Please sign in to comment.