Skip to content

Made changes to support the quantikz package#2843

Merged
lervag merged 1 commit intolervag:masterfrom
ArvinSKushwaha:master
Nov 30, 2023
Merged

Made changes to support the quantikz package#2843
lervag merged 1 commit intolervag:masterfrom
ArvinSKushwaha:master

Conversation

@ArvinSKushwaha
Copy link
Copy Markdown
Contributor

To resolve issue #2842, I have followed the format of the commits for issue #2755 to both correctly classify the quantikz environment (for the quantikz package) and test the changes.

lervag added a commit that referenced this pull request Nov 30, 2023
@lervag lervag merged commit 0eb8cc3 into lervag:master Nov 30, 2023
@lervag
Copy link
Copy Markdown
Owner

lervag commented Nov 30, 2023

Your code didn't work immediately because you forgot to give the function a name. Notice my diff that fixed it:

diff --git a/autoload/vimtex/syntax/p/quantikz.vim b/autoload/vimtex/syntax/p/quantikz.vim
index 82f9369a..5599a3c8 100644
--- a/autoload/vimtex/syntax/p/quantikz.vim
+++ b/autoload/vimtex/syntax/p/quantikz.vim
@@ -4,7 +4,7 @@
 " Email:      karl.yngve@gmail.com
 "
 
-function! vimtex#syntax#p#quantikz(cfg) abort " {{{1
+function! vimtex#syntax#p#quantikz#load(cfg) abort " {{{1
   call vimtex#syntax#core#new_env({
         \ 'name': 'quantikz',
         \ 'starred': v:false,

@ArvinSKushwaha
Copy link
Copy Markdown
Contributor Author

Ah, thank you!

@lervag
Copy link
Copy Markdown
Owner

lervag commented Nov 30, 2023

No problem :)

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