Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Match cursorSave and cursorRestore escape codes #45

Merged
merged 5 commits into from
Oct 28, 2021
Merged

Match cursorSave and cursorRestore escape codes #45

merged 5 commits into from
Oct 28, 2021

Conversation

SamVerschueren
Copy link
Contributor

fixes #36

I've added additional tests that iterates over all the ansi-escapes exported strings and functions. If it's a function, I just pass in some random data.

If you want to see something done differently, let me know 🙏 .

test.js Outdated Show resolved Hide resolved
fixtures/ansi-codes.js Outdated Show resolved Hide resolved
Co-authored-by: Qix <Qix-@users.noreply.github.com>
test.js Show resolved Hide resolved
Copy link
Member

@Qix- Qix- left a comment

Choose a reason for hiding this comment

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

Looks good. I'm a bit surprised the regex didn't cover these codes already.

Will defer to @sindresorhus.

test.js Show resolved Hide resolved
@sindresorhus sindresorhus merged commit 02fa893 into chalk:main Oct 28, 2021
Copy link

@jen801 jen801 left a comment

Choose a reason for hiding this comment

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

#46

Copy link

@jen801 jen801 left a comment

Choose a reason for hiding this comment

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

#test.js

@chalk chalk locked as resolved and limited conversation to collaborators Apr 1, 2022
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Can't match cursorSave and cursorRestore
4 participants