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

PR でビルドに関係ないドキュメント等の更新の場合に CI ビルドが走らないようにする #919

Merged
merged 1 commit into from
May 19, 2019

Conversation

m-tmatma
Copy link
Member

@m-tmatma m-tmatma commented May 19, 2019

PR の目的

PR でビルドに関係ないドキュメント等の更新の場合に CI ビルドが走らないようにする

カテゴリ

  • 仕様変更
  • CI関連
    • Azure Pipelines

PR の背景

#918 はドキュメントの更新で本来 CI のビルドが走る必要のない PR ですが、実際にはビルドが走っています。

branches に対してはビルドをスキップする指定が行われていますが、PR に対しては指定していなかったので、不必要にビルドが走っている。

PR のメリット

ドキュメントのみの更新で、CI の実行が必要ない PR のビルドをスキップできる。

PR のデメリット (トレードオフとかあれば)

特にない。

PR の影響範囲

md ファイル等を commit & push したときに PR で azure pipelines のビルドが走らなくなる。

関連チケット

#918

参考資料

https://docs.microsoft.com/en-us/azure/devops/pipelines/build/triggers?view=azure-devops&tabs=yaml#pull-request-validation

@m-tmatma
Copy link
Member Author

なお、Pull Request Template の branches に対するパスの除外指定に関しては #918 に含めることにして、この PR では外しています。

@m-tmatma m-tmatma added this to the v2.4.0 milestone May 19, 2019
@m-tmatma m-tmatma merged commit f7d1113 into sakura-editor:master May 19, 2019
@m-tmatma m-tmatma deleted the feature/exclude-pr-paths branch May 19, 2019 21:46
HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this pull request Jun 11, 2019
…-paths

PR でビルドに関係ないドキュメント等の更新の場合に CI ビルドが走らないようにする
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