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

[eme] Add polyfill tests #3364

Merged
merged 2 commits into from
Jul 30, 2016
Merged

Conversation

mwatson2
Copy link
Contributor

This PR is dependent on #3355

@hoppipolla-critic-bot
Copy link

Critic review: https://critic.hoppipolla.co.uk/r/6760

This is an external review system which you may optionally use for the code review of your pull request.

In order to help critic track your changes, please do not make in-place history rewrites (e.g. via git rebase -i or git commit --amend) when updating this pull request.

@wpt-pr-bot
Copy link
Collaborator

Reviewers for this pull request are: @ddorwin.

@jdsmith3000
Copy link
Contributor

I have no objections to merging these in, but recommend that each polyfill be discussed at the working group before committing to keep it.

I prefer to merge them now though to allow broader access to the tests, and to let us handle details using GitHub issues.

LGTM.

@mwatson2 mwatson2 merged commit 5dcad84 into web-platform-tests:master Jul 30, 2016
ddorwin pushed a commit to ddorwin/web-platform-tests that referenced this pull request Sep 13, 2016
* Add polyfill tests

* Fix lint issues
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants