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.

Sign up for GitHub

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

Jump to bottom

マージキューでだけ落ちるテストを追加してみる #14

Closed
wants to merge 3 commits into from
Closed

マージキューでだけ落ちるテストを追加してみる #14

wants to merge 3 commits into from
+12 −0

Conversation

sevenc-nanashi
Copy link
Member

@sevenc-nanashi sevenc-nanashi commented Jan 4, 2025 •
edited
Loading

内容

マージキューでだけ落ちるテストを追加してみます。
#13 が先。

関連 Issue

スクリーンショット・動画など

（なし）

その他

（なし）

@sevenc-nanashi sevenc-nanashi marked this pull request as draft January 4, 2025 00:53
@Hiroshiba
Copy link
Member

Hiroshiba commented Jan 4, 2025 •
edited
Loading

merge queueに入れれば良い･･･？

@sevenc-nanashi sevenc-nanashi marked this pull request as ready for review January 4, 2025 01:08
@sevenc-nanashi
Copy link
Member Author

#15 を待っていた感じですね。#15がマージされればいいはず。

Hiroshiba
Hiroshiba approved these changes Jan 5, 2025
View reviewed changes
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にするのが良さそう！

@sevenc-nanashi sevenc-nanashi added this pull request to the merge queue Jan 5, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jan 5, 2025
@sevenc-nanashi
Copy link
Member Author

sevenc-nanashi commented Jan 5, 2025 •
edited
Loading

お、落ちてくれた
というわけで、pushのbranchesからgh-readonly-queueを除外すれば1回だけテストが走るようになります。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Hiroshiba Hiroshiba Hiroshiba approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sevenc-nanashi @Hiroshiba