Skip to content

ci: add hl2sdk bump workflow#1

Merged
dollannn merged 4 commits intomainfrom
chore/add-hl2sdk-workflow
Dec 6, 2025
Merged

ci: add hl2sdk bump workflow#1
dollannn merged 4 commits intomainfrom
chore/add-hl2sdk-workflow

Conversation

@dollannn
Copy link
Member

@dollannn dollannn commented Dec 6, 2025

No description provided.

- name: Get commit details
if: steps.check.outputs.needs_update == 'true' && steps.existing_pr.outputs.skip != 'true'
id: commit_info
run: |
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

📝 [actionlint] reported by reviewdog 🐶
shellcheck reported issue in this script: SC2086:info:10:33: Double quote to prevent globbing and word splitting [shellcheck]

- name: Get commit details
if: steps.check.outputs.needs_update == 'true' && steps.existing_pr.outputs.skip != 'true'
id: commit_info
run: |
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

📝 [actionlint] reported by reviewdog 🐶
shellcheck reported issue in this script: SC2086:info:11:29: Double quote to prevent globbing and word splitting [shellcheck]

- name: Get commit details
if: steps.check.outputs.needs_update == 'true' && steps.existing_pr.outputs.skip != 'true'
id: commit_info
run: |
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

📝 [actionlint] reported by reviewdog 🐶
shellcheck reported issue in this script: SC2086:info:12:32: Double quote to prevent globbing and word splitting [shellcheck]

- name: Get commit details
if: steps.check.outputs.needs_update == 'true' && steps.existing_pr.outputs.skip != 'true'
id: commit_info
run: |
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

📝 [actionlint] reported by reviewdog 🐶
shellcheck reported issue in this script: SC2086:info:7:24: Double quote to prevent globbing and word splitting [shellcheck]

- name: Get commit details
if: steps.check.outputs.needs_update == 'true' && steps.existing_pr.outputs.skip != 'true'
id: commit_info
run: |
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

📝 [actionlint] reported by reviewdog 🐶
shellcheck reported issue in this script: SC2086:info:8:23: Double quote to prevent globbing and word splitting [shellcheck]

- name: Get commit details
if: steps.check.outputs.needs_update == 'true' && steps.existing_pr.outputs.skip != 'true'
id: commit_info
run: |
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

📝 [actionlint] reported by reviewdog 🐶
shellcheck reported issue in this script: SC2086:info:9:15: Double quote to prevent globbing and word splitting [shellcheck]

- name: Get commit details
if: steps.check.outputs.needs_update == 'true' && steps.existing_pr.outputs.skip != 'true'
id: commit_info
run: |
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [actionlint] reported by reviewdog 🐶
shellcheck reported issue in this script: SC2129:style:7:1: Consider using { cmd1; cmd2; } >> file instead of individual redirects [shellcheck]

if: steps.check.outputs.needs_update == 'true'
env:
GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
run: |
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

📝 [actionlint] reported by reviewdog 🐶
shellcheck reported issue in this script: SC2016:info:7:71: Expressions don't expand in single quotes, use double quotes for that [shellcheck]

@dollannn dollannn merged commit d4f316e into main Dec 6, 2025
7 checks passed
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.

1 participant

Comments