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

Update validation.md #223

Closed
wants to merge 4 commits into from
Closed

Update validation.md #223

wants to merge 4 commits into from

Conversation

neighborhood999
Copy link
Member

No description provided.

@chentsulin
Copy link
Member

這篇第二次 full conflict 了...應該歷史紀錄整個壞去

這樣 git 失效,下次還是會 full conflict,看是不是用某種方法,例如先讓它變回全英文,再拿那個去修?

@jigsawye
Copy link
Member

@chentsulin 先 commit 成原文最新版,再重新對照翻譯過一次這樣應該可行?
@appleboy 有法子嗎 @_@

@appleboy
Copy link
Member

先拿原文整個蓋過去吧,再把翻譯好的一部份一部份貼過去,不過通常這種整個 conflict 壞掉,感覺是編碼錯誤。

@chentsulin
Copy link
Member

👍

@neighborhood999
Copy link
Member Author

@chentsulin @appleboy 我先把原文覆蓋上去了,等 merge 後我再重新將他貼上。Thanks!

@chentsulin
Copy link
Member

應該不用先 merge ? 有這個 commit 應該就可以貼了?是嗎 @appleboy

@appleboy
Copy link
Member

直接用這邊的編輯功能就可以修改 PR 了

@appleboy
Copy link
Member

或者是直接修改你自己的 local branch,修改好之後重新 push 覆蓋,就可以更動此 PR 了

@neighborhood999
Copy link
Member Author

@chentsulin @appleboy 我已經把它覆蓋了,但有部分是新的,有做翻譯,再麻煩各位 review 了,謝謝。

@jigsawye
Copy link
Member

jigsawye commented Nov 2, 2015

直接 commit 回原文之後再重新翻譯過 8411a05

@jigsawye jigsawye closed this Nov 2, 2015
@neighborhood999 neighborhood999 deleted the try-update branch December 9, 2015 17:24
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