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

Cthulhu7th success notation #232

Merged
merged 3 commits into from
Jul 6, 2020

Conversation

AdmiralNyar
Copy link
Contributor

新クトゥルフの成功度の表記が紛らわしかったので、ルールブックの表記に修正

(パルプ・クトゥルフのダイスボットもクラスを継承しているので、パルプ・クトゥルフのテストデータも修正を加えた)

@ysakasin
Copy link
Member

ysakasin commented Jul 6, 2020

@AdmiralNyar 意見がブレてすみませんが、やっぱり ファンブル、失敗、レギュラー成功、ハード成功、イクストリーム成功、クリティカル にしませんか? 日本語版の翻訳に合わせるのも正しいのですが、「の」があるとあまりにも見づらいのと、これまでの日本におけるクトゥルフ神話TRPGの慣例として「クリティカル成功」や「クリティカルの成功」は馴染みがなさすぎると思います。

「クリティカル」についてはルールブックp.84では「クリティカル」「クリティカルでの成功」、p.104には「クリティカルの結果では〜」などと書かれていますし、十分根拠があるかと思いました。

@ysakasin
Copy link
Member

ysakasin commented Jul 6, 2020

  • 「の」がなくても誤解の発生しない表記であること
  • 日本語訳に一言一句合わせる必要性が低いこと

の二点が理由です

@ysakasin
Copy link
Member

ysakasin commented Jul 6, 2020

私家訳から変更した時のPR #120

@AdmiralNyar
Copy link
Contributor Author

「の」の件、了解しました。

作業していて思ったのですが、新クトゥルフからクラスを継承している、パルプ・クトゥルフのテストデータに新クトゥルフとの重複部分のテスト(CC,CBR,FARの部分)がだいぶ多いのですが、重複部分のテストデータを消すというのはダメなのでしょうか?
(また、別のPR出した方がいいでしょうか?)

@ysakasin
Copy link
Member

ysakasin commented Jul 6, 2020

作業していて思ったのですが、新クトゥルフからクラスを継承している、パルプ・クトゥルフのテストデータに新クトゥルフとの重複部分のテスト(CC,CBR,FARの部分)がだいぶ多いのですが、重複部分のテストデータを消すというのはダメなのでしょうか?
(また、別のPR出した方がいいでしょうか?)

新クトゥルフとテストケースを全く同じにする必要はないです。
継承したコマンドがそもそも動くのかが最低限確認できるテストケースがあれば十分かと思います。

パルプ・クトゥルフのテストケース削減は別Pull Requestでお願いいたします。

@AdmiralNyar
Copy link
Contributor Author

ありがとうございます
「の」の修正しましたので、確認お願いいたします

@ysakasin ysakasin merged commit 966a71b into bcdice:master Jul 6, 2020
@AdmiralNyar AdmiralNyar deleted the Cthulhu7th_success_notation branch July 6, 2020 14:42
ysakasin pushed a commit that referenced this pull request Aug 30, 2020
* 成功度の表記をルールブックにあわせる

* 成功度の表記をルールブックにあわせる(テストデータの修正)

* 「の」の修正
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