w33ble/forgejo-cli
Watch
1
0
Fork
You've already forked forgejo-cli
 0
Code Issues Pull requests Releases Packages Activity Actions

ci: run shellcheck on pull requests #4

Merged
w33ble merged 3 commits from feature/ci-shellcheck into main 2026-09-01 11:52:41 -06:00
Conversation 3 Commits 3 Files changed 1 +21
hijynx commented 2026-09-01 10:56:10 -06:00
Collaborator
Copy link

Adds .forgejo/workflows/shellcheck.yml: a Forgejo Actions workflow that lints forgejo and hooks/pre-commit on every PR (and pushes to PR branches), matching the local pre-commit hook's scope.

  • runs-on: docker (default docker-mode runner label; comment in the file explains what to change if the runner registered a different label)
  • actions/checkout@v4 + apt-installed shellcheck
  • This PR is its own acceptance test: the pull_request trigger fires from the merge ref, so the workflow runs here
Adds `.forgejo/workflows/shellcheck.yml`: a Forgejo Actions workflow that lints `forgejo` and `hooks/pre-commit` on every PR (and pushes to PR branches), matching the local pre-commit hook's scope. - `runs-on: docker` (default docker-mode runner label; comment in the file explains what to change if the runner registered a different label) - `actions/checkout@v4` + apt-installed shellcheck - This PR is its own acceptance test: the pull_request trigger fires from the merge ref, so the workflow runs here
ci: run shellcheck on pull requests
All checks were successful
shellcheck / shellcheck (pull_request) Successful in 32s
Details
e5f9ef5005
w33ble requested changes 2026-09-01 11:41:47 -06:00
Dismissed
.forgejo/workflows/shellcheck.yml Outdated
@ -0,0 +11,4 @@
# (node:20-bookworm image, apt available). If the job stays pending,
# the runner registered a different label — check `forgejo-runner
# list-labels` and set it here.
runs-on: docker
w33ble commented 2026-09-01 11:41:43 -06:00
Owner
Copy link

run on ubuntu-latest instead

run on `ubuntu-latest` instead
ci: run shellcheck job on ubuntu-latest label
All checks were successful
shellcheck / shellcheck (pull_request) Successful in 32s
Details
a79c088149
w33ble requested changes 2026-09-01 11:50:30 -06:00
Dismissed
w33ble left a comment
Copy link

ditch the comments, they are not helpful

ditch the comments, they are not helpful
.forgejo/workflows/shellcheck.yml Outdated
@ -0,0 +1,25 @@
# Shellcheck on every PR (and every push to a PR branch).
# Job runs from the PR merge ref, so this file is exercised by its own PR.
w33ble commented 2026-09-01 11:50:04 -06:00
Owner
Copy link

these comments are noise, remove them

these comments are noise, remove them
.forgejo/workflows/shellcheck.yml Outdated
@ -0,0 +8,4 @@
jobs:
shellcheck:
# If the job stays pending, the runner registered different labels —
# check `forgejo-runner list-labels` and set it here.
w33ble commented 2026-09-01 11:49:49 -06:00
Owner
Copy link

these comments are noise, remove them

these comments are noise, remove them
ci: remove workflow comments
All checks were successful
shellcheck / shellcheck (pull_request) Successful in 33s
Details
4b99dad6ab
w33ble approved these changes 2026-09-01 11:52:36 -06:00
w33ble merged commit 4b99dad6ab into main 2026-09-01 11:52:41 -06:00
Sign in to join this conversation.
Reviewers
No reviewers
w33ble
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
w33ble/forgejo-cli!4
No description provided.