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

ブランチ名に関わらずAZP/GHAが実行されるようにする #1788

Merged
1 commit merged into from Feb 7, 2022
Merged

ブランチ名に関わらずAZP/GHAが実行されるようにする #1788

1 commit merged into from Feb 7, 2022

Conversation

ghost
Copy link

@ghost ghost commented Feb 5, 2022

PR の目的

実行対象ブランチの名称に関わらずCIが実行されるようにします。

カテゴリ

  • ビルド関連
    • Azure Pipelines
    • GitHub Actions

PR の背景

サクラエディタが利用しているCIのうち、Azure PipelinesとGitHub Actionsでは、あらかじめ想定されるパターンに合致する名称のブランチに対してのみジョブが実行されます。
しかし、この方法ではブランチにつけられる名前に制限が課されるため、自由に名前を付けられず不便です。

過去には想定外のブランチ名でPRが出されたことがあり、それらのPRは事前の確認が不十分な可能性があります。

PR のメリット

  • ブランチ名に関わらず常にすべてのCIが実行されるようになるので、確実にビルド成否を確認できます。
  • ブランチ名に利用できる命名パターンの制約がなくなり、自由に名前をつけられるようになります。

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

仕様・動作説明

イベントトリガーの条件欄からブランチ名の記述を削除します。

PR の影響範囲

  • Azure Pipelines
  • GitHub Actions
    • GHA上でSonarCloudを実行するジョブは影響を受けません。

テスト内容

  • このブランチから派生した想定外の名前のブランチを作成し、remoteにpushしてCIが実行されることを確認します。

関連 issue, PR

※このPRは #1787 から分割されたものです。

参考資料

@ghost
Copy link
Author

ghost commented Feb 5, 2022

分割しました。反応次第で撤回されることに変わりはないです。
ご意見お待ちしております。

@ghost ghost mentioned this pull request Feb 5, 2022
@sonarcloud
Copy link

sonarcloud bot commented Feb 5, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@k-takata
Copy link
Member

k-takata commented Feb 5, 2022

よいと思います。

Copy link
Contributor

@berryzplus berryzplus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTMです。

ビルド対象ブランチを制限する仕組みは、
サンプルに書いてあったから入っただけなはずです。

指定外ブランチを利用したテストが時々行われていましたが、よくよく考えてみると、ビルドを除外したいくらいに遅いのはAppveyorだけなので、
他CIは無制限で良いのかもしれないです。

branches:
- master
- feature/*
- release/*
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ここ1年くらいリリースしてないですね。 > @KENCHjp さん

@ghost ghost marked this pull request as ready for review February 6, 2022 02:37
@ghost
Copy link
Author

ghost commented Feb 6, 2022

レビューありがとうございます。
他に意見がなければ近日中にマージしてしまいます。

@ghost ghost added azure pipelines GitHub Actions GitHub Actions関連 labels Feb 6, 2022
@ghost ghost merged commit d6212e8 into sakura-editor:master Feb 7, 2022
@ghost
Copy link
Author

ghost commented Feb 7, 2022

マージしました。ご確認ありがとうございました。

@ghost ghost deleted the feature/change_ci_trigger_on_any_branches branch February 8, 2022 22:11
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants