[GITHUB] Only perform actions to a bug if branch is main, master, or a release branch. Ignore feature, bugfix branches, etc.
Categories
(bugzilla.mozilla.org :: API, defect)
Tracking
()
People
(Reporter: jonalmeida, Assigned: dkl)
References
Details
(Keywords: requested-mobile)
Attachments
(1 file)
In bug 1821511, a change was made from a branch in the upstream project mobile-mobile/firefox-android
.
Actual results:
On creating the pull request, the linked bug was immediately closed.
Expected results:
The bug is closed only once merged.
Assignee | ||
Comment 1•2 years ago
|
||
(In reply to Jonathan Almeida [:jonalmeida] from comment #0)
In bug 1821511, a change was made from a branch in the upstream project
mobile-mobile/firefox-android
.Actual results:
On creating the pull request, the linked bug was immediately closed.Expected results:
The bug is closed only once merged.
Ok this might be serious so I may turn off support until this is fixed.
So to be clear, we should only be closing bugs if the commit is on either master
or a release branch such as releases_v110
?
Also, curious, was the pull request from a branch directly on the repository or from separate fork?
Thanks
Comment 2•2 years ago
|
||
Only commits to main should close the bug. Commits to releases_vXXX should set the appropriate status flags and that's it.
Reporter | ||
Comment 3•2 years ago
|
||
So to be clear, we should only be closing bugs if the commit is on either
master
or a release branch such asreleases_v110
?
What Ryan said, main
.
Also, curious, was the pull request from a branch directly on the repository or from separate fork?
Yes, in the linked github issue, it was created from a branch directly on the upstream repository, not the fork.
Assignee | ||
Comment 4•2 years ago
|
||
(In reply to Ryan VanderMeulen [:RyanVM] from comment #2)
Only commits to main should close the bug. Commits to releases_vXXX should set the appropriate status flags and that's it.
Ok understood about only closing bugs for commits to main. I was under the impressing
Comment 5•2 years ago
|
||
Updated•2 years ago
|
Assignee | ||
Comment 6•2 years ago
|
||
Description
•