-
-
Notifications
You must be signed in to change notification settings - Fork 4.5k
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
Add jquery.turbolinks v2.1.0 #7604
Conversation
with npm auto-update config added, close cdnjs#7254
@dannyxx001 Please help me review this PR, thank you. |
{ | ||
"basePath": "", | ||
"files": [ | ||
"vendor/assets/javascripts/*" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think you can use "vendor/assets/javascripts/" as basepath and "jquery.turbolinks*" as files in npmFileMap, but I'm not very sure. @PeterDaveHello Would you please explain again that whether only "dist" and "build" can be basepath? Thank you.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In this case, vendor/assets/javascripts/
or vendor/assets/
could be the base path, maybe you can discuss with its author and see if it's possible add a dist
folder for release files?
Any updates here? Thanks! |
2 similar comments
Any updates here? Thanks! |
Any updates here? Thanks! |
close cdnjs#7604, close cdnjs#7254, cc cdnjs#11170
- Add repository info - Update authors and keywords cc cdnjs#7604, cdnjs#7254, cdnjs#11170
PR for #7254
Checklist for Pull request or lib adding request issue follows the conventions.
Note that if you are using a distribution purpose repository/package, please also provide the url and other related info like popularity of the source code repo/package.
Profile of the lib
Essential checklist
Auto-update checklist
Git commit checklist
with npm auto-update config added, close #7254