Skip to content

Match dependabot.yaml files in language server - #2026

Merged
woodruffw merged 1 commit into
zizmorcore:mainfrom
fionn:dependabot-yaml-suffix
May 16, 2026
Merged

woodruffw merged 1 commit into
zizmorcore:mainfrom
fionn:dependabot-yaml-suffix

Conversation

@fionn

@fionn fionn commented May 16, 2026

Copy link
Copy Markdown
Contributor

The language server understands that Dependabot files match **/.github/dependabot.{yml,yaml}, however later on we check for dependabot.yml explicitly, thus excluding dependabot.yaml files.

We add dependabot.yaml as a file name to match against in the same manner as our matches for action.yml or action.yaml.

This has been tested in Neovim with the standard nvim-lspconfig settings, using

vim.lsp.config("zizmor", {
    cmd = {"/path/to/zizmor/target/debug/zizmor", "--lsp"}
})

to override the executable.

Closes: #2025


Pre-submission checks

Please check these boxes:

  • Mandatory: This PR corresponds to an issue (if not, please create
    one first).

  • Having read the AI policy, I hereby disclose the use of an LLM or other
    AI coding assistant in the creation of this PR. PRs will not be rejected
    for using AI tools, but will be rejected for undisclosed use or
    use that violates the policy.

If a checkbox is not applicable, you can leave it unchecked.

@woodruffw woodruffw added bugfix Fixes a known bug language-server labels May 16, 2026
The language server understands that Dependabot files match
**/.github/dependabot.{yml,yaml}, however later on we check for
dependabot.yml explicitly, thus excluding dependabot.yaml files.

We add dependabot.yaml as a file name to match against in the same
manner as our matches for action.yml or action.yaml.

This has been tested in Neovim with the standard nvim-lspconfig
settings, using

    vim.lsp.config("zizmor", {
        cmd = {"/path/to/zizmor/target/debug/zizmor", "--lsp"}
    })

to override the executable.

Closes: #2025
@woodruffw

Copy link
Copy Markdown
Member

Thank you @fionn!

@woodruffw
woodruffw enabled auto-merge (squash) May 16, 2026 18:20
@woodruffw
woodruffw merged commit 4f70a67 into zizmorcore:main May 16, 2026
12 checks passed
@fionn
fionn deleted the dependabot-yaml-suffix branch May 16, 2026 19:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix Fixes a known bug language-server

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG]: dependabot.yaml files are not recognised as such by the language server

2 participants