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

Revert "fix: stringify error on 'Cannot convert a Symbol value to a string'" #3022

Closed
wants to merge 1 commit into from

Conversation

johnjbarton
Copy link
Contributor

Reverts #2990

The tests fail on IE-win7 and win10. because the platform does not support "Symbol".

I do not understand why the tests passed before and github does not seem to keep the test results after a merge 👎

@Florian-R
Copy link

What about wrapping the test in a

if (window.Symbol)

instead of reverting?

@johnjbarton
Copy link
Contributor Author

Yes that would work if you want to send it.

@johnjbarton johnjbarton deleted the revert-2990-fix-stringify branch August 7, 2018 23:38
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