Add Mysql 5.7 json support

[bboure]

Add json support for mysql 5.7.

https://dev.mysql.com/doc/refman/5.7/en/json.html

[codecov-io]

Current coverage is 63.63% (diff: 100%)

Merging #172 into master will increase coverage by 2.55%

@@             master       #172   diff @@
==========================================
  Files            25         25          
  Lines          2960       2772   -188   
  Methods         323        323          
  Messages          0          0          
  Branches          0          0          
==========================================
- Hits           1808       1764    -44   
+ Misses         1152       1008   -144   
  Partials          0          0          

Powered by Codecov. Last update d8bfc6b...841bf81

[kevcam4891]

Wow. I had no idea MySQL supported the JSON type, and I've been using it
everyday for 8 years! :D

Benoit Boure mailto:notifications@github.com
October 26, 2016 at 9:40 AM

Add json support for mysql 5.7.

https://dev.mysql.com/doc/refman/5.7/en/json.html

---

    You can view, comment on, or merge this pull request online at:

#172

    Commit Summary

• Add Mysql 5.7 json support

  File Changes
  

• M lib/Ruckusing/Adapter/MySQL/Base.php
  https://github.com/ruckus/ruckusing-migrations/pull/172/files#diff-0
  (7)

  Patch Links:
  

• https://github.com/ruckus/ruckusing-migrations/pull/172.patch

• https://github.com/ruckus/ruckusing-migrations/pull/172.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#172, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AEb3dncgsDtjPzSkx1zUJl8ScFizIFl4ks5q31g4gaJpZM4KhLor.

[bboure]

This PR has been open for almost 4 years now without merging. I guess I'll close it

[ruckus]

Hi @bboure thanks for reaching out. I have not touched this project in many years and have essentially abandoned it. I don't have a PHP stack locally I could setup to test this PR. But a quick glance shows that it should work :)

If you want we can re-open and I'll accept the PR.

[bboure]

Thanks @ruckus for getting back. I have not used this project for a long time either, but we have been using a fork at Vreasy for years that included this change. It has been working great ever since.

I'll reopen it if you want to merge it

Thanks.

[ruckus]

Thank you!