Skip to content
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

Change zh.yml to zh-CN.yml #590

Merged
merged 2 commits into from
Sep 19, 2015
Merged

Change zh.yml to zh-CN.yml #590

merged 2 commits into from
Sep 19, 2015

Conversation

ethanyoung
Copy link
Contributor

zh is not valid in Rails I18n.

See details in #588.

jonatack added a commit that referenced this pull request Sep 19, 2015
@jonatack jonatack merged commit 8ae479f into activerecord-hackery:master Sep 19, 2015
@jonatack
Copy link
Contributor

Thanks!

@jonatack
Copy link
Contributor

In the future, 2 suggestions:

  • Squash your commits down to one single commit for the git history. Not a big deal here on Ransack, but on a project like Rails, for example, they will ask you to squash before merging.
  • When your PR fixes an issue, there’s one more little trick that makes it even more awesome: Add Closes #XXX. as the last line of your commit message. This automatically closes the issue when your PR is merged in 💯

@ethanyoung
Copy link
Contributor Author

Thanks for the suggestions!

On 20 Sep 2015 7:17 am, Jon Atack [email protected] wrote:
In the future, 2 suggestions:

  • Squash your commits down to one single commit for the git history. Not a big deal here on Ransack, but on a project like Rails, for example, they will ask you to squash before merging.
  • When your PR fixes an issue, there’s one more little trick that makes it even more awesome: Add Closes #XXX. as the last line of your commit message. This automatically closes the issue when your PR is merged in 💯

Reply to this email directly or view it on GitHub:
#590 (comment)

jonatack added a commit that referenced this pull request Sep 20, 2015
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.

2 participants