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

Translate expect vs should syntax to zh-tw #160

Conversation

gaga5lala
Copy link

Thanks for this awesome guide :)

In this PR, fix a typo and translate "expect vs should syntax" to zh-tw

@gaga5lala gaga5lala changed the title Translate expect vs should syntax to zh tw Translate expect vs should syntax to zh-tw Jan 9, 2016
@@ -258,7 +259,7 @@
</p> -->

<p>
「單一條件」意指一個測項應該只帶有一個檢查 (expection, assertion)。這樣做能幫助你直接前往失敗的測項尋找可能的問題,也讓程式碼比較好看。
「單一條件」意指一個測項應該只帶有一個檢查 (expectation, assertion)。這樣做能幫助你直接前往失敗的測項尋找可能的問題,也讓程式碼比較好看。

Choose a reason for hiding this comment

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

why this change?

The 'one expectation' tip is more broadly expressed as 'each test should make only one assertion'. This helps you on finding possible errors, going directly to the failing test, and to make your code readable.
http://betterspecs.org/#single

Copy link
Author

Choose a reason for hiding this comment

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

Thanks, for your review :)

I can't find "expection" in oxford dictionary, so I think it's should be noun for expect, so I change this to "expectation"
http://www.oxfordlearnersdictionaries.com/spellcheck/english/?q=expection

Maybe I misunderstood it's means?

Choose a reason for hiding this comment

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

I think this could be translated into:

「單一條件(one expectation)」泛指一個測項應該只有一個檢查(assertion)。

@gaga5lala gaga5lala force-pushed the translate-expect-vs-should-syntax-to-zh-tw branch from b4cab07 to 9cfda49 Compare January 10, 2016 09:19
@gaga5lala
Copy link
Author

Hi @JuanitoFatas , thanks for all your suggestions :)

Follow your suggestions, I push 2 new commit to fix them, one only thing I haven't change is keeping comment out original text.

@JuanitoFatas
Copy link

LGTM.

@onebree
Copy link
Contributor

onebree commented Jan 11, 2016

@JuanitoFatas thank you for reviewing this translation PR. And @gaga5lala thank you for your fixes!

onebree added a commit that referenced this pull request Jan 11, 2016
…tax-to-zh-tw

Translate expect vs should syntax to zh-tw
@onebree onebree merged commit 7728669 into betterspecs:master Jan 11, 2016
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.

None yet

3 participants