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

Resolve the babel rc relative to the file path, not the CWD. Fixes #252 #253

Merged
merged 1 commit into from
Aug 7, 2016

Conversation

lukeapage
Copy link
Contributor

See #252 for explanation of problem.

Would be great if you can take a look, I need this so am happy to make any change to get it in.

@lukeapage
Copy link
Contributor Author

@hzoo any chance you can take a look at this - it seems to me a straight forward bugfix.
Thanks.

@lukeapage
Copy link
Contributor Author

@hzoo @loganfsmyth
Any chance you can take a look at this fix and let me know if I need to do something to get it included?

@danez danez added the bug label Aug 2, 2016
@joshwiens
Copy link
Contributor

@lukeapage - Can you get this rebased and up to date so we can get it reviewed and landed?

@lukeapage
Copy link
Contributor Author

@d3viant0ne rebased, now up to date

@joshwiens
Copy link
Contributor

Resolves #252

@joshwiens joshwiens merged commit fca4b92 into babel:master Aug 7, 2016
@joshwiens
Copy link
Contributor

Thanks @lukeapage

Ognian pushed a commit to Ognian/babel-loader that referenced this pull request Aug 8, 2016
Resolve the babel rc relative to the file path, not the CWD Fixes babel#252
Ognian pushed a commit to Ognian/babel-loader that referenced this pull request Feb 27, 2017
Resolve the babel rc relative to the file path, not the CWD Fixes babel#252
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.

None yet

3 participants