Azure Pipelinesの除外条件を更新する #1791
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
berryzplus
approved these changes
Feb 9, 2022
berryzplus
reviewed
Feb 9, 2022
| @@ -6,32 +6,11 @@ | |||
| trigger: | |||
| paths: | |||
| exclude: | |||
| - .github/* | |||
| - '**/*.md' | |||
| - '.github/*' | |||
There was a problem hiding this comment.
参考。引用符で括った場合と括らない場合で挙動に差はなかった記憶です。
| - .gitignore | ||
| - .travis.yml |
There was a problem hiding this comment.
参考。サクラエディタは travis-CI でのビルドを想定してないです。
パクってきたテンプレに書いてあったから誤って混入してた認識です。
導入時に突っ込んで黙認した記憶があります。
削ってしまって問題ないはずです。
| - .gitignore | ||
| - .travis.yml | ||
| - .editorconfig | ||
| - appveyor.yml |
There was a problem hiding this comment.
参考。これは残しておくのが正しい（これで問題ない）の認識です。
| - tools/macro/macro.md | ||
| - tools/zip/readme.md | ||
| - tests/unittest.md | ||
| - vcx-props/project-PlatformToolset.md |
There was a problem hiding this comment.
参考。この変更により、すべてのmarkdownをここに列挙しなくても良くなります。
- 追加や削除、変更があったときにCI設定を修正しなくてよくなるということです。
- markdownをソースにしてプログラムに情報を埋め込む仕様は、採用しにくくなるということです。
需要があるかどうかは別にして。
|
たぶんこういうことっす。 ■実行環境×除外設定マトリクス
■ターゲット×除外設定マトリクス
|
気づくだろうと思ったので敢えて書きませんでしたが、このPRでは環境ごとに違えるべき条件以外を「全環境で統一する」のが目的ですので、各CIの設定ファイルを変更しても無視されることはありません。 |
|
レビューありがとうございます。 |
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 の目的
実行条件を更新し、いずれのCIも同じ条件でトリガーされるようにする。
カテゴリ
PR の背景
Azure Pipelinesの実行条件のうち、除外ファイルの指定内容がほかのCIと異なっているため、編集したファイルによっては必要ないにも関わらずCIが実行されてしまいます。
他のCIに揃える形で条件を統一し、確実に抑止されるようにします。
PR のメリット
PR のデメリット (トレードオフとかあれば)
仕様・動作説明
かつてAzure Pipelinesはワイルドカードに対応していなったため、 #1502 で対象ファイルを個別に列挙していますが、この方法では新たに除外対象とすべきファイルがリポジトリに追加されたときに更新を忘れる可能性があります。
幸い昨年9月にワイルドカードへの対応が実施されたので、他と同様にワイルドカードで指定するように改めます。
また、.editorconfigについてもAzure Pipelinesだけは除外対象となっておらず、代わりにTravis CIの設定ファイルが除外対象とされていることから、前者の追加と後者の削除を行います。
現時点でTravis CIは導入されておらず、当該設定ファイルも存在しませんので、除外から外しても問題はないはずです。
PR の影響範囲
テスト内容
あらかじめ ./installer/Languages/README.md を編集して、すべてのCIが実行されないことを確認済みです。
なお、このファイルは現時点で除外対象から漏れています。
関連 issue, PR
#1494 → #1498 → #1500 → #1501 → #1502
#1787
参考資料
Support for wild cards and conditional expressions in YAML pipeline files | Microsoft Docs