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

Swap out unicode ellipsis for ... #327

Merged
merged 1 commit into from
Dec 13, 2022
Merged

Swap out unicode ellipsis for ... #327

merged 1 commit into from
Dec 13, 2022

Conversation

sritchie
Copy link
Collaborator

@sritchie sritchie commented Dec 9, 2022

The unicode ellipsis is not a valid js character. If I do an :advanced build with

:pseudo-names true
:pretty-print true

The resulting JS fails to load, and I see

image

in the console. Making this change fixes the error.

@sritchie sritchie requested a review from mk December 9, 2022 16:49
@mk mk merged commit 5bb69d0 into main Dec 13, 2022
@mk mk deleted the sritchie-patch-1 branch December 13, 2022 14:53
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