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
Fix/improve es3 compat #2305
Fix/improve es3 compat #2305
Conversation
I'll squash my contribution |
d52f2d5
to
9ac4a2d
Compare
Looks good to me. |
Any reason
? |
Would they be used in-browser in any way? |
@ScottFreeCode maybe headlessly. |
@ScottFreeCode in the future you can simply rebase the branch onto |
b9e9631
to
add08ad
Compare
I do generally rebase my own branches rather than merging updates; in this case I just wasn't sure if I should mess with ndhoule's commit. Either way, looks like this is ready to roll? (Or as ready as it will be once we work out any unrelated Windows/AppVeyor kinks... EDIT: Actually, looks like AppVeyor's not passing simply because it's turned on for the project but hasn't been added to this code yet at all.) |
add08ad
to
d7c459b
Compare
see #2259 for original PR.