Skip to content

Initial implementation#1

Merged
omus merged 14 commits intomainfrom
cv/initial-action
Jan 16, 2025
Merged

Initial implementation#1
omus merged 14 commits intomainfrom
cv/initial-action

Conversation

@omus
Copy link
Member

@omus omus commented Jan 9, 2025

Possibly a better name for this action would be workflow-run-context which mirrors the GHA job-context.

Update: I think I change my mind here as we're not getting the current workflow run context but rather getting details about another workflow run.

@github-actions
Copy link

sh-checker report

To get the full details, please check in the job output.

shellcheck errors

'shellcheck ' returned error 1 finding the following syntactical issues:

----------

In workflow_scripts/integration-tests.yaml/job=Test/step=Test.sh line 5:
[[ ":steps.workflow.outputs.run-id:" -eq ":github.run_id:" ]] || exit 1
   ^-- SC2309 (warning): -eq treats this as an arithmetic expression. Use = to compare as string (or expand explicitly with $((expr))).
                                         ^---------------^ SC2309 (warning): -eq treats this as an arithmetic expression. Use = to compare as string (or expand explicitly with $((expr))).


In workflow_scripts/integration-tests.yaml/job=Test/step=Test.sh line 6:
[[ ":steps.workflow.outputs.run-attempt:" -eq ":github.run_attempt:" ]] || exit 1
   ^-- SC2309 (warning): -eq treats this as an arithmetic expression. Use = to compare as string (or expand explicitly with $((expr))).
                                              ^--------------------^ SC2309 (warning): -eq treats this as an arithmetic expression. Use = to compare as string (or expand explicitly with $((expr))).

For more information:
  https://www.shellcheck.net/wiki/SC2309 -- -eq treats this as an arithmetic ...
----------

You can address the above issues in one of three ways:
1. Manually correct the issue in the offending shell script;
2. Disable specific issues by adding the comment:
  # shellcheck disable=NNNN
above the line that contains the issue, where NNNN is the error code;
3. Add '-e NNNN' to the SHELLCHECK_OPTS setting in your .yml action file.



shfmt errors
'shfmt ' found no issues.

@github-actions
Copy link

sh-checker report

To get the full details, please check in the job output.

shellcheck errors

'shellcheck ' returned error 1 finding the following syntactical issues:

----------

In workflow_scripts/integration-tests.yaml/job=Test/step=Test.sh line 9:
[[ "${run_id}" -eq ":github.run_id:" ]] || exit 1
                   ^---------------^ SC2309 (warning): -eq treats this as an arithmetic expression. Use = to compare as string (or expand explicitly with $((expr))).


In workflow_scripts/integration-tests.yaml/job=Test/step=Test.sh line 10:
[[ "${run_attempt}" -eq ":github.run_attempt:" ]] || exit 1
                        ^--------------------^ SC2309 (warning): -eq treats this as an arithmetic expression. Use = to compare as string (or expand explicitly with $((expr))).

For more information:
  https://www.shellcheck.net/wiki/SC2309 -- -eq treats this as an arithmetic ...
----------

You can address the above issues in one of three ways:
1. Manually correct the issue in the offending shell script;
2. Disable specific issues by adding the comment:
  # shellcheck disable=NNNN
above the line that contains the issue, where NNNN is the error code;
3. Add '-e NNNN' to the SHELLCHECK_OPTS setting in your .yml action file.



shfmt errors
'shfmt ' found no issues.

@omus omus requested a review from kleinschmidt January 15, 2025 17:35
@omus omus merged commit 3e10123 into main Jan 16, 2025
4 checks passed
@omus omus deleted the cv/initial-action branch January 16, 2025 22:07
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