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

ci: リリース CI の実行条件を修正 #180

Merged
merged 2 commits into from
May 12, 2022
Merged

Conversation

MikuroXina
Copy link
Collaborator

Issue: #140

Type of Change:

Ci ワークフローの修正

Cause of the Problem (問題の原因)

リリースをあらゆる main への push にしていたためいくつかの弊害があった

Dealing with Problems (問題への対処)

リリース CI の実行条件を, v*.*.* の形式のタグが push されたときに変更しました.

Additional Information (追加情報)

これ以降, リリースするには以下のようにコマンドを実行するか, ほか git クライアントでタグを作成 & push することになります.

# バージョン 1.2.3 の場合
git tag v1.2.3
git push origin v1.2.3

@MikuroXina MikuroXina self-assigned this May 12, 2022
@MikuroXina MikuroXina requested a review from m1sk9 May 12, 2022 06:03
@codecov-commenter
Copy link

Codecov Report

Merging #180 (cd2eb1d) into main (22f5a2c) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #180   +/-   ##
=======================================
  Coverage   51.40%   51.40%           
=======================================
  Files          35       35           
  Lines         749      749           
  Branches      122      122           
=======================================
  Hits          385      385           
  Misses        245      245           
  Partials      119      119           
Flag Coverage Δ
unittests 51.40% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 22f5a2c...cd2eb1d. Read the comment docs.

Copy link
Contributor

@m1sk9 m1sk9 left a comment

Choose a reason for hiding this comment

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

変更については特にありませんが、追加情報を見る限り現状のコントリビューターにも影響がありそうなので、CONTRIBUTING.md にも同時にこの内容の記載を行いたいです。(このブランチに取り込む方向でいいかな?)

@MikuroXina
Copy link
Collaborator Author

MikuroXina commented May 12, 2022

限り現状のコントリビューターにも影響がありそうなので、CONTRIBUTING.md にも同時にこの内容の記載を行いたいです。

@mk-el 影響があるのは私達メンテナだけでしょう. 私達がリリースしたいタイミングでタグの追加をすることになります.

@m1sk9
Copy link
Contributor

m1sk9 commented May 12, 2022

限り現状のコントリビューターにも影響がありそうなので、CONTRIBUTING.md にも同時にこの内容の記載を行いたいです。

@mk-el 影響があるのは私達メンテナだけでしょう. 私達がリリースしたいタイミングでタグの追加をすることになります.

@MikuroXina, あ、なるほど。
そうですね、CONTRIBUTING.md に記述はいらなそう

approveします

@MikuroXina MikuroXina merged commit a1d4a1c into main May 12, 2022
@MikuroXina MikuroXina deleted the ci/fix-release branch May 12, 2022 11:50
@github-actions
Copy link
Contributor

🎉 This PR is included in version 1.12.4 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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.

3 participants