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

feat(markdown-and-latex): install markmap from mason #258

Merged
merged 1 commit into from
Jun 4, 2023
Merged

feat(markdown-and-latex): install markmap from mason #258

merged 1 commit into from
Jun 4, 2023

Conversation

jay-babu
Copy link
Contributor

@jay-babu jay-babu commented Jun 4, 2023

No description provided.

@mehalter
Copy link
Member

mehalter commented Jun 4, 2023

@jay-babu does mason-null-ls support ensuring the installation of this?

@jay-babu
Copy link
Contributor Author

jay-babu commented Jun 4, 2023

@jay-babu does mason-null-ls support ensuring the installation of this?

it is trick (or bug depending on your perspective) that I have no plans fixing :). mason-null-ls does not enforce any checks to ensure the requested tool is in null-ls registry

@mehalter
Copy link
Member

mehalter commented Jun 4, 2023

Ah I see 😂 I didn't know if it would require you to use a name that would be translated to a mason name. Cool thanks! I'll merge this!

Copy link
Member

@mehalter mehalter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@mehalter mehalter merged commit abd7777 into AstroNvim:main Jun 4, 2023
4 checks passed
@jay-babu
Copy link
Contributor Author

jay-babu commented Jun 4, 2023

Ah I see 😂 I didn't know if it would require you to use a name that would be translated to a mason name. Cool thanks! I'll merge this!

Simply converts - to _ and back. Other characters are unaffected

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

2 participants