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

Sync with reactjs.org @ f767be60 #118

Merged
merged 6 commits into from
Feb 13, 2019
Merged

Sync with reactjs.org @ f767be60 #118

merged 6 commits into from
Feb 13, 2019

Conversation

reactjs-translation-bot
Copy link
Contributor

@reactjs-translation-bot reactjs-translation-bot commented Feb 13, 2019

This PR was automatically generated.

Merge changes from reactjs.org at f767be6

The following files have conflicts and may need new translations:

  • content/docs/lists-and-keys.md

DO NOT SQUASH MERGE THIS PULL REQUEST!

Doing so will "erase" the commits from master and cause them to show up as conflicts the next time we merge.

@smikitky
Copy link
Member

Note: This will conflict with #115 but I'll take care of #115 after this has been merged

@smikitky smikitky mentioned this pull request Feb 13, 2019
@smikitky
Copy link
Member

@koba04 これはやっぱり master を一旦マージせずに直接マージコミット作る方が正しそうな気がします…

@koba04
Copy link
Member

koba04 commented Feb 13, 2019

@smikitky syncの場合はそうですね。この場合は、無視してmergeしちゃっていいかなと思います。squashしない点も他とは異なりますし。

@smikitky smikitky merged commit e00e9a0 into master Feb 13, 2019
@smikitky
Copy link
Member

"admin権限を使って強制的にマージする" みたいなチェックボックスをチェックする必要がありましたが、マージはできました

@smikitky smikitky deleted the sync-f767be60 branch February 13, 2019 05:52
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.

5 participants