Skip to content

Update jQuery#41

Open
shoyan wants to merge 2 commits intotkengo:masterfrom
shoyan:update-jQuery
Open

Update jQuery#41
shoyan wants to merge 2 commits intotkengo:masterfrom
shoyan:update-jQuery

Conversation

@shoyan
Copy link

@shoyan shoyan commented Jan 16, 2015

JQuery update from 2.0.1 to 2.1.3.

@keitakawamoto
Copy link
Collaborator

LGTM 👍

@tkengo
Copy link
Owner

tkengo commented Jan 16, 2015

karmaで各種ファイルを読み込んでいる部分で古いjQueryを読みに行っててテストが落ちてるみたいなので、こっちも修正してちょ。

https://github.com/tkengo/hometype/blob/master/karma.conf.js#L6

@tkengo
Copy link
Owner

tkengo commented Jan 16, 2015

あとマニフェストファイルで読み込んでる部分もだね。
https://github.com/tkengo/hometype/blob/master/manifest.json#L30

@tkengo
Copy link
Owner

tkengo commented Jan 16, 2015

オプションページの読み込み部分もよろ〜

<script type="text/javascript" src="../lib/jquery-2.0.1.min.js"></script>

@shoyan
Copy link
Author

shoyan commented Jan 26, 2015

なるほど。了解す〜!

@keitakawamoto
Copy link
Collaborator

👀

],
"js": [
"lib/jquery-2.0.1.min.js",
"lib/jquery-2.1.3.min.js",
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

daff

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tes

],
"js": [
"lib/jquery-2.0.1.min.js",
"lib/jquery-2.1.3.min.js",
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

gagd

Copy link
Owner

@tkengo tkengo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

あだ

@tkengo tkengo self-requested a review March 16, 2022 08:08
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.

3 participants