Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(breadcrumbs): remove Markdown from excluded file types #3196

Conversation

Nefendi
Copy link
Contributor

@Nefendi Nefendi commented Oct 10, 2022

Description

The Markdown entry in the list of excluded file types for the breadcrumbs module did nothing, as the output of vim.bo.filetype for MD files yields the value of markdown.

How Has This Been Tested?

No tests other than the simple smoke ones were necessary.

Additional remarks

This PR addresses the discussion in #3189.

@ChristianChiarulli ChristianChiarulli marked this pull request as ready for review October 16, 2022 00:25
@ChristianChiarulli
Copy link
Member

pls fix conflicts

@Nefendi Nefendi force-pushed the remove-dead-markdown-entry-in-breadcrumbs-ignore-file-types-list branch from cdd6be4 to f71595b Compare October 16, 2022 08:36
@Nefendi
Copy link
Contributor Author

Nefendi commented Oct 16, 2022

done

@Nefendi
Copy link
Contributor Author

Nefendi commented Oct 16, 2022

BTW, I have initially marked this PR as a draft because that is what CONTRIBUTING.md states:

To avoid duplicate work, create a draft pull request.

The output of `vim.bo-filetype` for Markdown files is 'markdown' not
'Markdown' so this entry did nothing.
@Nefendi Nefendi force-pushed the remove-dead-markdown-entry-in-breadcrumbs-ignore-file-types-list branch from f71595b to 7ab5ba1 Compare October 22, 2022 11:14
@cpea2506 cpea2506 changed the title chore(breadcrumbs): remove dead Markdown entry from excluded file types chore(breadcrumbs): remove Markdown from excluded file types Nov 20, 2022
@cpea2506
Copy link
Member

@ChristianChiarulli I think it's now okay to be merged

@ChristianChiarulli
Copy link
Member

but it's pointed at rolling

@cpea2506 cpea2506 changed the base branch from rolling to master November 21, 2022 04:51
@cpea2506
Copy link
Member

but it's pointed at rolling

My bad, fixed it. 😅

@cpea2506
Copy link
Member

Thanks 🚀

@cpea2506 cpea2506 merged commit 38ddcb4 into LunarVim:master Nov 21, 2022
@Nefendi Nefendi deleted the remove-dead-markdown-entry-in-breadcrumbs-ignore-file-types-list branch November 21, 2022 14:26
tomazursic pushed a commit to tomazursic/LunarVim that referenced this pull request Nov 23, 2022
* upstream/master:
  chore: bump plugins version (LunarVim#3506)
  docs: github pull request template update (LunarVim#3512)
  fix(uninstaller): correct a sentence in help (LunarVim#3511)
  fix(uninstaller): add separate flag to remove user config (LunarVim#3508)
  chore: bump plugins version (LunarVim#3483)
  chore(breadcrumbs): remove `Markdown` from excluded file types (LunarVim#3196)
  chore: update docker branch to latest release for "Try it first" (LunarVim#3500)
  fix: discard invalid choice when closing buffers (LunarVim#3488)
  fix(nvimtree): avoid hard-coded mapping (LunarVim#3492)
  fix(autocmds): remove spell autocmd (LunarVim#3487)
  fix(lsp): sumneko-lua library scanning (LunarVim#3484)
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.

None yet

3 participants