-
Notifications
You must be signed in to change notification settings - Fork 12
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
Install via npm and using it with webpack fails #5
Comments
@tedbarth Thank you for your detail reporting. I'll check it as soon as possible. |
SolutionI find your solution. use instead of Reason for FailsBecause we did not provide Line 67 in c9a0bd2
But fortunately we provided it on
Plan
If you have any issue, then please comment again. |
Problem solved. Thanks for the very fast fix! Didn't expect that! 🥇 |
I recently started using hammer.js. I like the simple API and it worked out of the box together with webpack. Sadly I got some issues with Firefox and after browsing the original GitHub project I saw that no one has released a new version since 2 years. I was happy to find your fork via this issue then.
I managed to install it via npm using the following command after uninstalling the original hammerjs dependency:
$ npm install git+https://git@github.com/naver/hammer.js.git --save
Which led to the following entry in my package.json:
"dependencies": { "@egjs/hammerjs": "git+https://git@github.com/naver/hammer.js.git", [...] }
I use your hammerjs exactly as I did before. I only prefixed it with @egjs as my IntelliJ idea (my IDE) suggested (it can find your code and complete symbols from it as it could do for the original hammerjs installed from here). Following code I do use:
Now, when running
npm run build
(which only starts webpack:"build": "webpack"
) I get the following error:The complete log from last line of the previous snippet contains:
My webpack config looks like this:
I hope you can tell me that it is just a simple fix to make your project run with webpack as it did before with the original one. Thanks in advance.
The text was updated successfully, but these errors were encountered: