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

export flow types in build #401

Closed
wants to merge 1 commit into from
Closed

export flow types in build #401

wants to merge 1 commit into from

Conversation

nmn
Copy link

@nmn nmn commented Jun 5, 2016

Updated the build script to export the source files with the .flow extension in the dist folder so that anyone using flow gets the type definitions for free.

Fixes #400

echos the path to all the test files as side-effect for now. I'm sure someone better at bash will be able to fix it.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 98.287% when pulling 9f54bb9 on nmn:flow-types into 359ec76 on graphql:master.

@KyleAMathews
Copy link

👍
On Sat, Jun 4, 2016 at 10:43 PM Coveralls notifications@github.com wrote:

[image: Coverage Status] https://coveralls.io/builds/6466956

Coverage remained the same at 98.287% when pulling 9f54bb9
9f54bb9
on nmn:flow-types
into 359ec76
359ec76
on graphql:master
.


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#401 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/AAEVh-VKvkWFj2s50kOiZNVz-dc7UO8sks5qImIIgaJpZM4IuSUW
.

@nmn
Copy link
Author

nmn commented Jun 30, 2016

Closing in favor of #412
Thank @nodkz

@nmn nmn closed this Jun 30, 2016
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.

Make flow types available by including source file in NPM release.
4 participants