search

NetCommons3 / NetCommons3

Other
38 stars 16 forks source link

travis-ciでMySQL5.7テスト #1265

Open akagane99 opened 6 years ago

akagane99 commented 6 years ago

https://github.com/NetCommons3/NetCommons3/issues/1262

レンサバがMySQL5.7を採用し始めていて（ロリポップ、さくらのレンタルサーバー等）、NC3のお知らせ歯車ボタン押下や、会員管理を開いたらエラーになってます。 travis-ciテストでMySQL5.7をやる方向がいいのかなぁ。

travis-ciでの設定方法は下記みたい。 https://docs.travis-ci.com/user/database-setup/#MySQL-5.7

'.travis.yml'の設定、既存のテストを残しつつ、どう設定してくのがいいんだろう？ ちと思いつかず、とりあえず報告まで。

RyujiAMANO commented 6 years ago

んー今はstrict mode に対応したクエリ修正かけられる余力のある開発メンバーがいなさそうだから、 今はやらない に +1

先にstrict mode 無効化修正かなぁ

RyujiAMANO commented 6 years ago

'.travis.yml'の設定、既存のテストを残しつつ、どう設定してくのがいいんだろう？ ちと思いつかず、とりあえず報告まで。

別ブランチ(feature/mysql5.7)とか切ってそちらだけ設定っていうのありかも。 でもってエラーなくなったらmasterにマージ。 エラーありのままで、masterに修正はいったら、masterからブランチに修正をマージ。

って書いてみておもったが、これを全プラグインでやるのはやっぱ大変だな^^; やっぱり手を動かせるようになってからやるのがよさそう

akagane99 commented 6 years ago

別ブランチ(feature/mysql5.7)

これいいですね。+1

そして、たしかに手動かせない(汗)

調査と対応は、別けて対応でもいいかも、と思いました。 ほとんど @RyujiAMANO さんと同じ内容です。 ・調査はこっち、全プラグイン、別ブランチ(feature/mysql5.7)切って、mysql5.7テストする。 ・対応はこっち、別ブランチに修正コミット。修正おわったらmasterマージ。

とりあえずここまで。

akagane99 commented 6 years ago

https://github.com/NetCommons3/NetCommons3/issues/1262#issuecomment-404504732

上記で対応したため、とりあえずmysql5.7のテストはしなくても大丈夫になりました。 とくになければ、2,3日位でクローズしようと思います。