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

Fix: Dictionary error in amsmath.vim #2814

Closed
wants to merge 1 commit into from

Conversation

DrJupiter
Copy link

I was encountering an error, when loading the plugin. To my understanding, it was in the declaration of the dictionary. Changing it to valid(?) syntax fixed my error.

I was encountering an error, when loading the plugin.
To my understanding, it was in the declaration of the dictionary.
Changing it to valid(?) syntax fixed my error.
@lervag
Copy link
Owner

lervag commented Aug 11, 2024

I'm terribly sorry that I've completely forgotten to respond to this PR! I don't understand how that happened!

So, your PR is necessary because the syntax I've used depends on the literal dict syntax which was introduced in Vim 8.1.1876. At the time you opened the PR, I should have accepted it since I specifically depended only on version 8.0.1453. However, the version requirement has since been bumped up to 8.2.3995. So, I won't accept this PR now, because I will be using more modern syntaxes and features.

However, if you are restricted to use an older Vim version, then I might consider to patch the v2.15 tag. Let me know if this is necessary for you.

@lervag lervag closed this Aug 11, 2024
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.

2 participants