-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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 support for backticks in reporter error messages #3608
Comments
Blocked by #3629 which bumps snap-shot-it which fixes a bug with escaped backticks. Tried a workaround with 89566e9 replacing backticks with single quotes, but think we should just upgrade. Tried figuring out the failing tests on the upgrade and couldn't figure it out, @bahmutov said he'd give it a go if he has time. |
This is now unblocked by snapshot-it 🎉 but now blocked because the snapshot-it changes are gonna every saved snapshot and some of them are failing for me locally on develop ( |
We can look at the failing ones together
…Sent from my iPhone
On Mar 21, 2019, at 19:26, Lila Conlee ***@***.***> wrote:
This is now unblocked by snapshot-it 🎉 but now blocked because the snapshot-it changes are gonna every saved snapshot and some of them are failing for me locally on develop (test/e2e/8_reporters_spec.coffee) which I'll see about getting around tomorrow.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
|
The code for this is done in cypress-io/cypress#3694, but has yet to be released. |
Closing as part of #3762 |
Current behavior:
No markdown support currently, linkify support added in #3457
Desired behavior:
Include support for backticks and escaping so we can use them in error messages.
The text was updated successfully, but these errors were encountered: