Skip to content

ci(lint): pin zsh-lint v1.4.0 and retire the lint pilot - #43

Merged
ss-o merged 1 commit into
mainfrom
ci/zsh-lint-1-4-0
Oct 10, 2026
Merged

ss-o merged 1 commit into
mainfrom
ci/zsh-lint-1-4-0

Conversation

@ss-o

@ss-o ss-o commented Oct 10, 2026

Copy link
Copy Markdown
Member

Moves the required Zsh Lint / Zsh Lint job from zsh-lint v1.3.0 to v1.4.0, and retires the shared-workflow pilot (zsh-lint-pilot.yml and .github/lint-pilot.md). The pilot ran the same analyzer pin in observe mode next to the enforcing job; its qualification (#30) is closed.

Part of z-shell/zsh-lint#612. zsh-lint v1.4.0: https://github.com/z-shell/zsh-lint/releases/tag/v1.4.0 (signed tag on aff6018). Building it needs Go 1.26, so the Go version now comes from the analyzer's own go.mod instead of a hardcoded "1.25".

v1.4.0 behavior that can show up here: quoting/unquoted-var is now Info and reports only standalone expansions, and rules now see builtin/command-prefixed commands (for example builtin eval).

Local check on this diff: zsh-lint --config zsh-lint.json build.zsh with v1.4.0 exits 0 with no findings. tests/release.zunit only checks the workflow triggers, which are unchanged.

Part of z-shell/zsh-lint#612: move this repository to the released zsh-lint v1.4.0.

@ss-o ss-o left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Self-review of record on 38a7f16.

Read the full diff; it matches the description. The verification listed in the description ran locally against this exact diff with a zsh-lint binary built from the v1.4.0 tag. actionlint passes on every changed workflow.

No blocking findings.

@ss-o
ss-o merged commit b0bd76b into main Oct 10, 2026
9 checks passed
@ss-o
ss-o deleted the ci/zsh-lint-1-4-0 branch October 10, 2026 12:32
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