Skip to content

Comments

Update MySQL server version and character set in Rails Guide [skip ci]#33929

Merged
kamipo merged 1 commit intorails:masterfrom
yahonda:update_utf8mb4_railsguide
Sep 20, 2018
Merged

Update MySQL server version and character set in Rails Guide [skip ci]#33929
kamipo merged 1 commit intorails:masterfrom
yahonda:update_utf8mb4_railsguide

Conversation

@yahonda
Copy link
Member

@yahonda yahonda commented Sep 20, 2018

Summary

This pull request updates MySQL server version and character set in Rails Guide [skip ci]
Related to #33853

@rails-bot
Copy link

r? @sgrif

(@rails-bot has picked a reviewer for you, use r? to override)

@yahonda
Copy link
Member Author

yahonda commented Sep 20, 2018

This is my first pull request to Rails Guide. Would like to get feedback from the technical and English grammar point of views.

Copy link
Member

Choose a reason for hiding this comment

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

I think "Rails 6 drops supporting MySQL 5.1" part is unnecessary for the guide, since we also didn't mention about which Rails version supports MySQL 5.0.

Copy link
Member Author

Choose a reason for hiding this comment

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

Make sense. Updated and made some additional changes.

@yahonda yahonda force-pushed the update_utf8mb4_railsguide branch from c428772 to abba020 Compare September 20, 2018 12:39
@kamipo kamipo merged commit 0a829f7 into rails:master Sep 20, 2018
@yahonda yahonda deleted the update_utf8mb4_railsguide branch September 23, 2018 05:05
yahonda added a commit to yahonda/rails that referenced this pull request Oct 30, 2018
rails#33853 and rails#33929 removed `encoding: utf8mb4` from database.yml
since at that time MySQL 5.1 is supported with the master branch.

Since MySQL 5.1 has been dropped, we can restore `encoding: utf8mb4` in database.yml
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.

4 participants