Skip to content

ci(lint): pin zsh-lint v1.4.0 - #236

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 Configured Zsh Lint job from zsh-lint v1.3.0 to v1.4.0.

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: v1.4.0 with --config zsh-lint.json public/zsh/init.zsh exits 0 with no findings.

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 66ad9d6.

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.

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Oct 10, 2026 •

Copy link
Copy Markdown

Deploying src with  Cloudflare Pages  Cloudflare Pages

Latest commit: 66ad9d6
Status: ✅  Deploy successful!
Preview URL: https://6a33f3a5.zi-src.pages.dev
Branch Preview URL: https://ci-zsh-lint-1-4-0.zi-src.pages.dev

View logs

@ss-o
ss-o merged commit 7e3fbaa into main Oct 10, 2026
16 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