Closed
Bug 1109162
Opened 10 years ago
Closed 10 years ago
Do not validate commit names for non-landable patches
Categories
(Firefox OS Graveyard :: Gaia::GithubBot, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: kgrandon, Assigned: kgrandon)
References
Details
Attachments
(1 file)
See bug 1042673.
It looks like Autolander did some validations on commits and pull request titles that did not have the R+. We should probably not perform these validations if the pull request is not landing (does not have an R+).
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → kgrandon
Status: NEW → ASSIGNED
Comment 1•10 years ago
|
||
Assignee | ||
Comment 2•10 years ago
|
||
Comment on attachment 8533967 [details] [review]
[PullReq] KevinGrandon:bug_1109162_do_not_validate_non_landable to mozilla:master
Just a simple movement of code, and all tests pass, so r=me.
Attachment #8533967 -
Flags: review+
Assignee | ||
Updated•10 years ago
|
Keywords: checkin-needed
Updated•10 years ago
|
Keywords: checkin-needed
Comment 3•10 years ago
|
||
https://github.com/mozilla/autolander/pull/25
The pull request could not be applied to the integration branch. Please try again after current integration is complete.
Assignee | ||
Comment 4•10 years ago
|
||
Oops, I guess that's what happens when Autolander doesn't have access to the repository.
Keywords: checkin-needed
Comment 5•10 years ago
|
||
Pull request has landed in master: https://github.com/mozilla/autolander/commit/704eaa2262b987f6dacec9b70899358dfcc015cb
Updated•10 years ago
|
Keywords: checkin-needed
Updated•10 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•