Skip to content

Handle commits with missing prs during landing setup#2696

Merged
lutien merged 2 commits intomozilla:masterfrom
lutien:handle-commits-with-missing-prs-during-landing-setup
Jan 22, 2026
Merged

Handle commits with missing prs during landing setup#2696
lutien merged 2 commits intomozilla:masterfrom
lutien:handle-commits-with-missing-prs-during-landing-setup

Conversation

@lutien
Copy link
Member

@lutien lutien commented Jan 21, 2026

No description provided.

@lutien lutien force-pushed the handle-commits-with-missing-prs-during-landing-setup branch from b9a3d13 to 31b030d Compare January 21, 2026 13:55
@lutien lutien marked this pull request as ready for review January 21, 2026 14:15
@lutien lutien requested a review from jgraham January 21, 2026 14:21
@lutien lutien force-pushed the handle-commits-with-missing-prs-during-landing-setup branch from eb9425c to 9478809 Compare January 21, 2026 17:07
@lutien
Copy link
Member Author

lutien commented Jan 21, 2026

I've just realized that I have read the code wrong, and we actually have to abort even if the landing is not created yet. But I guess it's still fine if we just return the error message in the log. The PR is updated.

Copy link
Member

@jgraham jgraham left a comment

Choose a reason for hiding this comment

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

Thanks! LGTM with some minor fixups.

@lutien lutien force-pushed the handle-commits-with-missing-prs-during-landing-setup branch from eb8c832 to bc04384 Compare January 22, 2026 13:03
@lutien lutien merged commit f174372 into mozilla:master Jan 22, 2026
4 checks passed
@lutien lutien deleted the handle-commits-with-missing-prs-during-landing-setup branch January 22, 2026 13:09
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

Comments