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

Local build errors #103

Open
arizonatribe opened this issue Jul 16, 2018 · 1 comment
Open

Local build errors #103

arizonatribe opened this issue Jul 16, 2018 · 1 comment

Comments

@arizonatribe
Copy link

arizonatribe commented Jul 16, 2018

I cloned the repo and started playing with the code locally but I ran into some problems just getting it to build. Perhaps this isn't causing a problem for anyone currently, but I suspect if you were to remove the node_modules/ folder and globally remove your versions of babel-eslint you might encounter the same problems I ran across.

It seems there was a bug in babel-eslint that doesn't play well with Flow, but it was corrected in version 8.1.1. For me it was causing the npm test command to fail. Anyway, a simple upgrade of that package fixed it.

Additionally, I ran into an error when building Spected that looked like a simple CommonJs failure to recognize the Object rest spread operator. I fixed it by moving the rollup commonjs() call (in the rollup.config.js) so that it comes after the babel() transpile step.

Hope this is helpful.

@busypeoples
Copy link
Owner

Thanks for the very valuable feedback @arizonatribe
Will try to reproduce the error in the coming days.

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

No branches or pull requests

2 participants