Skip to content

register the (arg)?mini[!*e]?, (arg)?maxi[!*]? environment from to region_math#2495

Merged
lervag merged 3 commits intolervag:masterfrom
huweiATgithub:support-math-optidef
Sep 26, 2022
Merged

register the (arg)?mini[!*e]?, (arg)?maxi[!*]? environment from to region_math#2495
lervag merged 3 commits intolervag:masterfrom
huweiATgithub:support-math-optidef

Conversation

@huweiATgithub
Copy link
Copy Markdown
Contributor

The package optidef provides mini, maxi, argmini, argmaxi environments that should be considered math environments.

Copy link
Copy Markdown
Owner

@lervag lervag left a comment

Choose a reason for hiding this comment

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

Thanks! This looks good and I don't see any immediate problems here. But I would kindly ask you to also add an example within test/test-syntax. Create two files, test-optidef.vim and test-optidef.tex. For this simple stuff you don't need to create any real tests, but having the example there makes it much easier for further development or bug fixing in the future in case something breaks.

@huweiATgithub
Copy link
Copy Markdown
Contributor Author

I added some simple tests.

@lervag lervag merged commit 0f0725d into lervag:master Sep 26, 2022
@lervag
Copy link
Copy Markdown
Owner

lervag commented Sep 26, 2022

Thank you!

@huweiATgithub huweiATgithub deleted the support-math-optidef branch September 27, 2022 06:08
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