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

Do not require es6 or downlevel-iterators TS flag #104

Merged
merged 1 commit into from
May 24, 2019

Conversation

matt-oakes
Copy link
Collaborator

Overview

This is a simple fix for #100 which removed the need for es6 or the downlevel-iterators TypeScript flag.

Fixes #100.

Test Plan

Tests should pass.

cc @mikehardy

@mikehardy
Copy link
Contributor

Cool - I was hoping it was a really small thing like this. Didn't seem like a huge deal but if it widens compatibility for such a small cost, maybe a worthwhile win. LGTM

@matt-oakes matt-oakes merged commit f91cdd3 into master May 24, 2019
react-native-community-bot pushed a commit that referenced this pull request May 24, 2019
## [3.1.2](v3.1.1...v3.1.2) (2019-05-24)

### Bug Fixes

* Do not require es6 or downlevel-iterators TS flag ([#104](#104)) ([f91cdd3](f91cdd3))
@react-native-community-bot
Copy link
Collaborator

🎉 This PR is included in version 3.1.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

@mikehardy
Copy link
Contributor

I AM SO JEALOUS of your CI and semantic-release setup. I really have to get that going for device-info. Thanks @matt-oakes !

@matt-oakes matt-oakes deleted the matt-oakes/fix-100 branch May 24, 2019 15:05
@matt-oakes
Copy link
Collaborator Author

I love that I can release new versions with the push of a button from bed 😉

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

Successfully merging this pull request may close these issues.

Typescript error / downlevel-iterators?
3 participants