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

Remove label checking from CI #448

Merged
merged 1 commit into from
May 18, 2024

Conversation

omochi
Copy link
Contributor

@omochi omochi commented May 18, 2024

現在、CI で正しいラベルがついているかチェックしています。
これにより、マージする前に正しいラベルをつけることを強制し、
リポジトリの運用をしやすくしているようです。

しかしコントリビュータにはラベルをつける権限がないので、
提出した PR が必ず失敗状態になってしまいます。

そのため、本来重要な 自動テストの失敗が隠蔽されるので、
プルリク一覧画面の見通しが悪くなってしまいます。

そして実際のところ、 @kateinoigakukun がマージした以下のPRは、

#438 #439 #440 #442 #443 #445

最終的にラベルが設定されないままマージされています。

これらは自動テストに成功していても、
ラベルチェックが失敗するので GitHub 上ではマージ後も失敗状態として表示されてしまっています。

ラベルを設定しないままマージすることを許容しているのであれば、
メリットを享受できていないですし、
デメリットしかないので、削除してもらえるとありがたいです。

@kateinoigakukun kateinoigakukun merged commit 14bb4f0 into swiftwasm:main May 18, 2024
1 of 2 checks passed
@kateinoigakukun kateinoigakukun changed the title CI からラベルチェックを取り除く Remove label checking from CI May 19, 2024
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