-
Notifications
You must be signed in to change notification settings - Fork 81
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
chore(dev-env): Remove bower on dev env #395
Conversation
Removed bower install for dev setup Updated all 'bower_components' reference Ref naver#391
Need to be checked
|
Update on cssPrefix jQuery version
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.
LGTM
@@ -6,7 +6,7 @@ | |||
<style> | |||
#el {font-size:20pt} | |||
</style> | |||
<script src="../../bower_components/jquery/jquery.js"></script> | |||
<script src="https://code.jquery.com/jquery-1.7.2.min.js"></script> |
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.
It seems to be slow beacuse get a network. and is different case. I think that be better merged jquery.
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.
Trying to figure out install multiple version of same package as dependency, but actually there's no appropriate solution for that.
Having a copy of legacy jQuery at unit/lib
seems to be best for now.
For cssPrefix only test purpose
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.
👍
Issue
#391
Details
Removed bower install for dev setup
Updated all 'bower_components' reference
Preferred reviewers
@naver/egjs-dev