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

Bump actions #798

Merged
merged 1 commit into from
Jun 4, 2024
Merged

Bump actions #798

merged 1 commit into from
Jun 4, 2024

Conversation

nix6839
Copy link
Contributor

@nix6839 nix6839 commented May 30, 2024

内容

すべてのアクションのバージョンを最新バージョンにアップグレードしました

その他

  • 他のアクションは問題ありませんが、github/issue-labeler@v3において以下のようなブレイキングチェンジが発生しましたが、無視しても問題ないでしょうか?

    Breaking change: Issue labels that do not match a regex will no longer be removed by default unless you set sync-labels: to 1

  • "PNGs"単語にタイポの問題が発生したため、無視するように設定しました。

@qryxip qryxip self-requested a review May 30, 2024 18:56
@nix6839
Copy link
Contributor Author

nix6839 commented Jun 3, 2024

_typos.tomlファイルのタブ文字(\t)を他のTOMLファイルと同じ4つのスペースに置き換えました。

Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

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

LGTM!!

Copy link
Member

@qryxip qryxip left a comment

Choose a reason for hiding this comment

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

LGTM!

build_java_packageのやつとかはむしろ多分今までのが壊れてましたね… 今は動かしていないので気付かなかった。

image

  • 他のアクションは問題ありませんが、github/issue-labeler@v3において以下のようなブレイキングチェンジが発生しましたが、無視しても問題ないでしょうか?

    Breaking change: Issue labels that do not match a regex will no longer be removed by default unless you set sync-labels: to 1

多分問題ないと思います。壊れたとしても、"labeler"自体廃止する方針なので。

@qryxip qryxip merged commit 264460d into VOICEVOX:main Jun 4, 2024
38 checks passed
@nix6839 nix6839 deleted the bump-actions branch June 4, 2024 13:11
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