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

追加: 非アクティブ issue へ自動的にコメント #1220

Merged
merged 7 commits into from
May 17, 2024

Conversation

tarepan
Copy link
Contributor

@tarepan tarepan commented May 7, 2024

内容

非アクティブ issue へ自動的にコメントする actions を追加

#1112, #1167 により非アクティブ issue は棚卸し(ロードマップ行き検討)することが決定した。このなかで棚卸し条件が明確化されたため、対象 issue を自動検出することが可能となった。
ところで、GitHub 公式の Action actions/stale は非アクティブ issue へのコメントと auto-close が可能である。これを利用すれば棚卸し対象 issue の管理を自動化できる。

このような背景から、非アクティブ issue へ自動的にコメントする actions の追加を提案します。

issue の auto-close は現段階で必要ないとの議論に基づき、対象 issue を自動検出してコメントを残す形としました。

関連 Issue

ref #1112, #1167

@tarepan tarepan requested a review from a team as a code owner May 7, 2024 10:16
@tarepan tarepan requested review from Hiroshiba and removed request for a team May 7, 2024 10:16
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.

とても良いですね!!!

いくつかめちゃくちゃ細かいコメントしました!!

.github/workflows/test-freshness.yml Outdated Show resolved Hide resolved
.github/workflows/test-freshness.yml Outdated Show resolved Hide resolved
.github/workflows/test-freshness.yml Outdated Show resolved Hide resolved
.github/workflows/test-freshness.yml Outdated Show resolved Hide resolved
.github/workflows/test-freshness.yml Outdated Show resolved Hide resolved
@tarepan
Copy link
Contributor Author

tarepan commented May 12, 2024

@Hiroshiba
全指摘箇所の反映・テストパスを確認しました。Re-review よろしくお願いします。

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!!

いくつか気になったので変更させていただきます!

.github/workflows/test-issue-freshness.yml Outdated Show resolved Hide resolved
.github/workflows/test-issue-freshness.yml Show resolved Hide resolved
@Hiroshiba Hiroshiba merged commit 3b1bf09 into VOICEVOX:master May 17, 2024
4 checks passed
@tarepan tarepan deleted the add/stale branch May 17, 2024 17:20
@tarepan
Copy link
Contributor Author

tarepan commented May 18, 2024

https://github.com/VOICEVOX/voicevox_engine/actions/runs/9136934313
30日チェックの動作を確認できた。
一方、180日チェックが動いていない可能性がある。any-of-labels の書式や日本語関係などを再確認する必要あり。

@Hiroshiba
Copy link
Member

通知便利ですね!!!実装感謝です!!

180日の方が動いてないのはラベルが付いてて180日経ってるのがないだけだったり…?

とりあえず忘れないようissue作るのもありなのかもと思いました!
もう作られていたらすみません🙇

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