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 API test for auto shim #16

Merged
merged 1 commit into from
Oct 22, 2017
Merged

Add API test for auto shim #16

merged 1 commit into from
Oct 22, 2017

Conversation

nolde
Copy link
Contributor

@nolde nolde commented Oct 14, 2017

This commit adds a test that will check for existence of auto.js.

This has been requested in es-shims/Promise.prototype.finally#12

- test will check for existence of `auto.js`
- fix typo
@nolde
Copy link
Contributor Author

nolde commented Oct 14, 2017

For some reason, I can't execute the tests locally, so I can't really debug the issue. That said, it doesn't seem to be related to my commit, but I may be wrong =) .

@ljharb
Copy link
Member

ljharb commented Oct 22, 2017

It's failing because of the linter; what happens when you run npm install && npm test locally?

@ljharb
Copy link
Member

ljharb commented Oct 22, 2017

(I've fixed it locally and pushed up the changes)

@ljharb ljharb merged commit 341401f into es-shims:master Oct 22, 2017
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.

None yet

2 participants