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 test for --demo flag #38

Merged
merged 3 commits into from
Sep 27, 2021
Merged

Conversation

msabramo
Copy link
Contributor

@msabramo msabramo commented Sep 15, 2021

I was working locally on a possible refactor and had the tests passing and thought I was good to go but then realized that my changes had broken --demo. 😢

1t5hik

test.js Outdated Show resolved Hide resolved
@sindresorhus sindresorhus changed the title Add test for --demo Add test for --demo flag Sep 17, 2021
@msabramo
Copy link
Contributor Author

Waiting on this one until #37 is resolved

@msabramo
Copy link
Contributor Author

msabramo commented Sep 27, 2021

#37 was merged, so resuming work on this. Switched to AVA snapshot testing in 7608084, as suggested by @sindresorhus above.

@sindresorhus sindresorhus merged commit b04fefc into chalk:main Sep 27, 2021
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