Skip to content

Commit

Permalink
Fix action lint in default ML workflow
Browse files Browse the repository at this point in the history
Fixes #2963
Co-authored-by: Maximilian Roos <5635139+max-sixty@users.noreply.github.com>
  • Loading branch information
nvuillam committed Nov 19, 2023
1 parent 6e4c4e9 commit a56cec9
Show file tree
Hide file tree
Showing 9 changed files with 10 additions and 9 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/auto-update-linters.yml
Original file line number Diff line number Diff line change
Expand Up @@ -108,7 +108,7 @@ jobs:

# Upload MegaLinter artifacts
- name: Archive production artifacts
if: ${{ success() }} || ${{ failure() }}
if: success() || failure()
uses: actions/upload-artifact@v3
with:
name: MegaLinter reports
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/deploy-DEV.yml
Original file line number Diff line number Diff line change
Expand Up @@ -225,7 +225,7 @@ jobs:

# Upload MegaLinter artifacts
- name: Archive production artifacts
if: ${{ success() }} || ${{ failure() }}
if: success() || failure()
uses: actions/upload-artifact@v3
with:
name: MegaLinter reports
Expand All @@ -236,7 +236,7 @@ jobs:
linter-versions.json
- name: debug
if: ${{ success() }} || ${{ failure() }}
if: success() || failure()
run: echo ${{ steps.docker_build.outcome }}

# Clean disk space
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/mega-linter-for-runner.yml
Original file line number Diff line number Diff line change
Expand Up @@ -106,7 +106,7 @@ jobs:
# Upload MegaLinter artifacts
- name: Archive production artifacts
uses: actions/upload-artifact@v3
if: ${{ success() }} || ${{ failure() }}
if: success() || failure()
with:
name: MegaLinter reports
path: |
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/mega-linter.yml
Original file line number Diff line number Diff line change
Expand Up @@ -106,7 +106,7 @@ jobs:
# Upload MegaLinter artifacts
- name: Archive production artifacts
uses: actions/upload-artifact@v3
if: ${{ success() }} || ${{ failure() }}
if: success() || failure()
with:
name: MegaLinter reports
path: |
Expand Down
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@ Note: Can be used with `oxsecurity/megalinter@beta` in your GitHub Action mega-l
- Update Pylint links to their current locations, by @echoix in [#3116](https://github.com/oxsecurity/megalinter/issues/3116)
- Fix way to install powershell on Alpine linux image
- Fix issue with VS Code devcontainer not building [#3114](https://github.com/oxsecurity/megalinter/issues/3114)
- Fix Default Workflow to handle latest ActionLint rules

- CI
- Upgrade pymdown-extensions and markdown, by @BryanQuigley in [#3053](https://github.com/oxsecurity/megalinter/pull/3053)
Expand Down
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -480,7 +480,7 @@ jobs:

# Upload MegaLinter artifacts
- name: Archive production artifacts
if: ${{ success() }} || ${{ failure() }}
if: success() || failure()
uses: actions/upload-artifact@v3
with:
name: MegaLinter reports
Expand Down
2 changes: 1 addition & 1 deletion TEMPLATES/mega-linter.yml
Original file line number Diff line number Diff line change
Expand Up @@ -101,7 +101,7 @@ jobs:
# Upload MegaLinter artifacts
- name: Archive production artifacts
uses: actions/upload-artifact@v3
if: ${{ success() }} || ${{ failure() }}
if: success() || failure()
with:
name: MegaLinter reports
path: |
Expand Down
2 changes: 1 addition & 1 deletion docs/install-github.md
Original file line number Diff line number Diff line change
Expand Up @@ -82,7 +82,7 @@ jobs:

# Upload MegaLinter artifacts
- name: Archive production artifacts
if: ${{ success() }} || ${{ failure() }}
if: success() || failure()
uses: actions/upload-artifact@v3
with:
name: MegaLinter reports
Expand Down
2 changes: 1 addition & 1 deletion docs/reporters/SarifReporter.md
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ If you want GitHub to handle the SARIF file to display it in its UI, add the fol

```yaml
- name: Upload MegaLinter scan results to GitHub Security tab
if: ${{ success() }} || ${{ failure() }}
if: success() || failure()
uses: github/codeql-action/upload-sarif@v2
with:
sarif_file: 'megalinter-reports/megalinter-report.sarif'
Expand Down

0 comments on commit a56cec9

Please sign in to comment.