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

allow dots in names of required and imported files #11

Merged
merged 1 commit into from
Jan 9, 2016

Conversation

AprilArcus
Copy link
Contributor

Version 0.3.0 throws a spurious error on e.g. require('./webpack.config') when ./webpack.config.js exists. This PR includes a fix and tests.

@mysticatea
Copy link
Owner

Oh, thank you for PRs!

I had not noticed that require adds .js even if there is a extension already.
I will merge this today (JST).

@AprilArcus
Copy link
Contributor Author

Thank you for the wonderful tool, and comprehensive test coverage. It was easy to jump right in and contribute. Your work sets an excellent example for the community.

mysticatea added a commit that referenced this pull request Jan 9, 2016
allow dots in names of required and imported files
@mysticatea mysticatea merged commit 615caa8 into mysticatea:master Jan 9, 2016
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

2 participants