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

Fixing babel transpiler errors. windows environments need to normaliz… #16

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nenitiko
Copy link

Tried running the generator in windows and got two different errors:

  1. babel-core was not added as a dev-dependency
  2. skipped files/folders in transpilationConfig (gulpfile.js) were not matching any files under windows due to the path separator differences between the Regex strings and the pathnames, this gives many babel transpilation errors for files in node_modules (jquery being one of the first)

I added both the babel-core npm depepdency and the normalization/conversion of backslashes to posix dir separators (by using github.com/sindresorhus/slash) before transpilation occurs.

…e directory separators otherwise babel transformation wont skip files specified in configuration
neelabhg added a commit to neelabhg/sysbuild that referenced this pull request Nov 22, 2015
1. Added babel-core as a dev-dependency.
2. Fixed path separator differences which were causing Babel to
transpile
skipped files under Windows.

The fix was taken from SteveSanderson/generator-ko#16.
@carbontwelve
Copy link

👍 This is ace, I spent half an hour tinkering and making it work then noticed this PR - you could have saved me half an hour had I looked before tinkering haha.

@gabriel-sas
Copy link

Good job in Fixing the issue! 👍

@jeanPokou
Copy link

thks, just fixed the bug

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

Successfully merging this pull request may close these issues.

None yet

6 participants