Skip to content

Update CI github action for versioning on local test.zip - #43

Closed
XenoS-ITA wants to merge 4 commits into
Tynopia:mainfrom
XenoS-ITA:main
Closed

XenoS-ITA wants to merge 4 commits into
Tynopia:mainfrom
XenoS-ITA:main

Conversation

@XenoS-ITA

Copy link
Copy Markdown
Contributor

No description provided.

@XenoS-ITA
XenoS-ITA requested a review from Tynopia as a code owner June 20, 2026 17:56
@XenoS-ITA

Copy link
Copy Markdown
Contributor Author

i runned super linter locally and the only issue seems to be that we're using node24 as the runner. It looks like something in the linter hasn't been updated to support it yet.

output from super linter

2026-07-09 17:40:55 [INFO]   Linting ENV items...
2026-07-09 17:40:55 [NOTICE]   ENV linted successfully
2026-07-09 17:40:55 [INFO]   Command output for ENV:
------
Checking .env.example

No problems found
------
2026-07-09 17:40:55 [INFO]   Linting GITHUB_ACTIONS items...
2026-07-09 17:40:55 [ERROR]   Found errors when linting GITHUB_ACTIONS. Exit code: 1.
2026-07-09 17:40:55 [INFO]   Command output for GITHUB_ACTIONS:
------
.github/workflows/ci.yml:74:9: shellcheck reported issue in this script: SC2129:style:10:1: Consider using { cmd1; cmd2; } >> file instead of individual redirects [shellcheck]
   |
74 |         run: |
   |         ^~~~
.github/workflows/ci.yml:106:15: invalid runner name "node24" at runs.using in "Upload Escrow Asset" action defined at "/tmp/lint". valid runners are "composite", "docker", and "node20". see https://docs.github.com/en/actions/creating-actions/metadata-syntax-for-github-actions#runs [action]
    |
106 |         uses: ./
    |               ^~
------
2026-07-09 17:40:55 [INFO]   Linting GITLEAKS items...
2026-07-09 17:40:57 [NOTICE]   GITLEAKS linted successfully
2026-07-09 17:40:57 [INFO]   Stderr contents for GITLEAKS:
------

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants