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

テストケースの表記ゆれ確認作業 #75

Merged
merged 4 commits into from
Oct 23, 2018
Merged

Conversation

SawadaStdDesign
Copy link
Contributor

WAIC-TEST-0001-01.txt 〜 WAIC-TEST-0027-06.txt の表記ゆれ確認/変更作業が完了しました。

@momdo
Copy link
Member

momdo commented Aug 24, 2018

@SawadaStdDesign これは、提案中のものも含めて、翻訳ガイドラインに寄せてみた、という認識であってますか?(もしそうなら、ガイドラインの記述不足か曖昧さがありそう、と思ったので聞いてみました。)

@SawadaStdDesign
Copy link
Contributor Author

SawadaStdDesign commented Aug 24, 2018

@momdo もともとは翻訳ガイドラインに則るための作業ではなく、テストケース内の表記ゆれを減らすという主旨の作業でしたが、作業しているうちに気になってきたので、何点か翻訳ガイドラインに則って変更した (つもりの) 部分があります。

・全角丸括弧→半角丸括弧 + 前後に半角スペース
・全角英数字→半角英数字 + 前後に半角スペース
・「」→“” (これは未定のもの) + 前後に半角スペース
・および→及び
・または→又は
・ユーザー→利用者

などです。
抜けがあるかも知れません。

@SawadaStdDesign
Copy link
Contributor Author

あ、コンフリクトが起きてるようですね... (対処方法が分かりません

@nishimotz
Copy link
Contributor

私の作業と衝突したかも?ちょっと見てみます

@nishimotz
Copy link
Contributor

notation-blur ブランチ「に」 master をマージしてみたら、衝突しませんでした。
この状態から pull request しなおせば大丈夫だと思います。

が、もしそちらでこの作業が難しいなら、私が別のブランチを作って PR をやりなおすしかないかな。。

@SawadaStdDesign
Copy link
Contributor Author

SawadaStdDesign commented Aug 24, 2018

@nishimotz 確認いただきありがとございます!
「pull request しなおせば」というのは

  1. pull して
  2. 上書きして
  3. commit して
  4. push して
  5. pull request する
    をもう一度やり直す、ということですね?
    その場合、この pull request に対しては何か処置は要らないんでしょうか? 消すとか閉じるとか

@nishimotz
Copy link
Contributor

nishimotz commented Aug 24, 2018

すでに GitHub で notation-blur ブランチが PR 75 (訂正済)と紐づいているから、
そちらからだったら notation-blur ブランチを更新して push しなおせば、
この PR の中身が新しくなるはずです。

@SawadaStdDesign
Copy link
Contributor Author

SawadaStdDesign commented Aug 24, 2018

@nishimotz
「notation-blur ブランチを更新」という操作が分からないんです...
「notation-blur ブランチに master をマージ」ということでしょうか?
(そうすればwipディレクトリが削除されてうまくいくような?)

SourceTreeに「マージ」ボタンがありますが、これでしょうかね?
SourceTreeの「マージ」ボタンを押した状態の画面キャプチャ

@SawadaStdDesign
Copy link
Contributor Author

SawadaStdDesign commented Aug 24, 2018

@nishimotz
それとも「リベース」という操作ですかね...?
SourceTreeの「リポジトリ」メニューを開いた状態の画面キャプチャ

@nishimotz
Copy link
Contributor

master をマージしてみてください。
「直ちにコミット」で大丈夫です。

成功すれば作業ファイルの notation-blur ブランチが
「master の最新コミットにさらに何かコミットしたもの」
になります。

念のために、提案したい変更がちゃんと残っていることを確認して、
GitHub にプッシュしてください。

@SawadaStdDesign
Copy link
Contributor Author

  1. notation-blur ブランチに master をマージして
  2. (ディレクトリは消えてなかったけど中身は消えてたので) もともとWIP内で更新していたファイルをWAIC-TESTに移動(上書き)して
  3. commit して
  4. push しました

どうやら「no conflicts」の模様です!

@nishimotz
Copy link
Contributor

LGTM

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