bug: doesn't attach when buffer has multiple filetypes, including one supported

Ouverte Adaptée aux débutants
#715 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Évaluation

Difficulté
2/5
Temps estimé
1-3 heures
Accessibilité débutants
78/100
Type d'issue
Bug
Clarté
Clairement spécifiée
Activité
Active
Stack technique
lua
Domaine
tooling

Piste de recherche

Commence par lire la comparaison des types de fichiers dans lua/render-markdown/core/manager.lua, vers la ligne 150, indiquée par l’issue. Vérifie comment le type de fichier du buffer est comparé aux types de fichiers configurés ; le travail est terminé lorsqu’un buffer dont le type de fichier est markdown.Avante est reconnu comme contenant markdown et que le plugin s’active.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

bug
Neovim version (nvim -v)

0.12.6

Neovim distribution

N/A

Operating system

nixos

Terminal emulator / GUI

kitty

Describe the bug

in https://github.com/avante-corp/avante.nvim, I want to assign the buffer multiple filetypes with set ft=markdown.Avante, as supported by vim ( see :h ft "When a dot appears in the value then this separates two filetype names, it should therefore not be used for a filetype. ").
This plugin does an exact string compareason at https://github.com/MeanderingProgrammer/render-markdown.nvim/blob/8e8795368391036643b622875a6da6a51bcd33f4/lua/render-markdown/core/manager.lua#L150 which fails when it compares "markdown" with "markdown.Avante". So render-markdown is not enabled while it should.

I believe nvim_get_option_value should return an array for "ft" but until it does, render-markdown could split the return filetypes on . and compare those

Expected behavior

avante should detect that ft contains the "markdown" and be enabled.

Healthcheck output
it's all green
Plugin configuration
require('render-markdown').setup({
    -- enabled = true,
    -- debounce = 100,
    log_level = 'error',
    log_runtime = false,
    -- to appease checkhealth
    html = { enabled = false },
    -- use recommended settings from above
    file_types = { 'markdown', 'Avante',
        -- 'markdown.Avante'  -- workaround
},
})
Plugin error log
empty
Confirmations
  • I have updated this plugin to the latest version using my plugin manager
  • I have updated all treesitter parsers (:TSUpdate)
  • I have provided the text contained in all screenshots as raw text in this issue. This means if there is a screenshot below it is the copy pasted contents of the file in the screenshot. I understand that my issue will be closed if I have not.
Additional information

No response

Langage dominant
Lua
Étoiles
5.2k
Forks
152
Métriques de merge des PR
Aucune PR mergée en 30 j

Préparer son environnement

Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de MeanderingProgrammer/render-markdown.nvim

Toutes les issues de MeanderingProgrammer/render-markdown.nvim

Issues similaires

Plus d'issues Lua

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.