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

Change \undertilde to \utilde #948

Merged
merged 4 commits into from
Nov 24, 2017
Merged

Change \undertilde to \utilde #948

merged 4 commits into from
Nov 24, 2017

Conversation

ronkok
Copy link
Collaborator

@ronkok ronkok commented Oct 24, 2017

In PR #670, I made an error. The function that should have been \utilde I named instead \undertilde.

There is an \undertilde from the wsuipa package, but it is a text-mode non-stretchy function. The proper command name is \utilde, a math-mode, stretchy function from packge undertilde.

This PR fixes my error.

In PR #670, I made an error. The function that should have been `\utilde` I named instead `\undertilde`.

There is an `\undertilde` from the `wsuipa` package, but it is a text-mode non-stretchy function. The proper command name is `\utilde`, a math-mode, stretchy function from packge `undertilde`.

This PR fixes my error.
@kevinbarabash
Copy link
Member

kevinbarabash commented Oct 24, 2017

Let's do a version bump to 0.11.x 0.10.x for this even though it's a bug fix so that people if anyone was using \undertilde it'll be less likely they'll be caught off guard by the change.

Copy link
Member

@edemaine edemaine left a comment

Choose a reason for hiding this comment

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

LGTM. @kevinbarabash Should we merge this into master now, or wait to non-alpha-release 0.9.0? Or, given that we're already changing the number to 0.9.0, maybe it's OK to include as part of 0.9.0?

@kevinbarabash
Copy link
Member

@edemaine good question. I was thinking of jumping to 0.10.0-alpha... we could also do 0.9.0 and just call out this breaking change and the one that was in 0.9.0-alpha.

@edemaine
Copy link
Member

edemaine commented Nov 21, 2017

@kevinbarabash Any objection to merging this to master? (now 1 month later) I don't want to overstep if you're about to release. We could do another 0.9.0 alpha/beta... (In any case, I think releasing is important to get the array bug fix out.)

Personally, I think changing this in 0.9.0 is fine.

@kevinbarabash kevinbarabash merged commit 75af19c into KaTeX:master Nov 24, 2017
@ronkok ronkok deleted the utilde branch November 24, 2017 16:48
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

3 participants