単体テストに関するドキュメントをサブディレクトリに移動する #1654
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ghost
added
the
beru
reviewed
May 2, 2021
tests/unittest.md
Outdated
| @@ -31,13 +31,14 @@ cmake を実行することにより、各プラットフォームに固有の | |||
|
|
|||
| googletest を使用して作成する単体テストも cmake でビルド設定を行います。 | |||
There was a problem hiding this comment.
コマンド名は全て小文字の cmake ですが固有名詞としては CMake なので記述を変えたほうが良いと思います。
とはいえ、cmake の見出し以降で全て小文字で書かれているので置き換えるとなると箇所が多いですね。
There was a problem hiding this comment.
ci/build-envvars.md にも「appveyor」のままになっている箇所が残っているので、
今度機会があるときに直しておきます。
There was a problem hiding this comment.
ci/build-envvars.md にも「appveyor」のままになっている箇所が残っているので、
今度機会があるときに直しておきます。
githash.bat の GitHub Actions 対応の時に行うドキュメント更新に含めるつもりでしたが、対応方法を見直した結果ドキュメントは更新不要と判断したので、この「機会」も当分来ないかもしれません。
beru
reviewed
May 2, 2021
tests/unittest.md
Outdated
| @@ -31,13 +31,14 @@ cmake を実行することにより、各プラットフォームに固有の | |||
|
|
|||
| googletest を使用して作成する単体テストも cmake でビルド設定を行います。 | |||
|
|
|||
| cmake は Visual Studio 2017 用のプロジェクト生成に対応しているため | |||
| Visual Studio 2017 用のプロジェクトを生成してビルドすれば | |||
| cmake は Visual Studio 用のプロジェクト生成に対応しているため | |||
There was a problem hiding this comment.
ここも同様にコマンド名は全て小文字の cmake ですが固有名詞としては CMake なので記述を変えたほうが良いと思います。
beru
approved these changes
May 2, 2021
|
レビューありがとうございます。とりあえずこれでマージします。 |
ghost
merged commit
This pull request was closed.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR の目的
リポジトリのルートディレクトリにある単体テストに関するドキュメントを、 tests ディレクトリに移します。
カテゴリ
PR の背景
既に Visual Studio 2019 でのビルドと、コンパイルテストの追加が行われましたが、ドキュメントに反映されていませんでした。
また、ルートディレクトリにあるファイルが多すぎるという指摘もありました。
PR のメリット
PR のデメリット (トレードオフとかあれば)
仕様・動作説明
PR の影響範囲
テスト内容
テスト1
関連 issue, PR
#698 Visual Studio 2019 対応作業（完了済）
#1297 コンパイルテストを導入
#740 ルートディレクトリのファイルを整理する（ CI/CD 関係のドキュメントは #1630 で実施済み。）
参考資料