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

Add diffcoeff and upgreek packages to command completion #3735

Merged
merged 8 commits into from
Nov 12, 2024

Conversation

Toseflo
Copy link
Contributor

@Toseflo Toseflo commented Nov 9, 2024

I added two new files for the diffcoeff and upgreek commands, as I use them regularly and would like to have command completion for them.

I'm not sure if I have to add something for handling of the commands or add something else. Please tell me if something is missing.

Also, I'm not quite sure what the collapse parameter does. I copied most of the upgreek file from LatexGreekCommand.kt, and it had collapse = true in every line. I don't know if it should stay there for upgreek or if I should remove it.

@PHPirates PHPirates self-requested a review November 10, 2024 12:31
@PHPirates
Copy link
Collaborator

Almost there :) just a few comments by Qodana and then should be good to go

@PHPirates PHPirates added this to the Next milestone Nov 12, 2024
Copy link
Collaborator

@PHPirates PHPirates left a comment

Choose a reason for hiding this comment

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

Nice, thanks for adding this!

@PHPirates PHPirates merged commit 83a0dd1 into Hannah-Sten:master Nov 12, 2024
9 checks passed
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