Skip to content

man: document that disable ranges are half-open - #3582

Open
ump45nose wants to merge 1 commit into
koalaman:masterfrom
ump45nose:man/disable-range-half-open
Open

ump45nose wants to merge 1 commit into
koalaman:masterfrom
ump45nose:man/disable-range-half-open

Conversation

@ump45nose

Copy link
Copy Markdown
Contributor

disable ranges are parsed into DisableComment from to, documented as [from, to) in src/ShellCheck/AST.hs:150, and matched with code >= from && code < to in src/ShellCheck/ASTLib.hs:764. So disable=SC3000-SC4000 covers SC3000 through SC3999 and leaves SC4000 enabled, which the existing "to exclude 3xxx" example does not make clear.

This adds one sentence to the disable documentation spelling out the exclusive end.

Docs-only change; no code or tests touched.

This branch has not been deployed

No deployments
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