19 Sep
2019
19 Sep
'19
9:07 p.m.
Kyle Stanley wrote:
Benjamin, what are you thoughts on usage of the "needs backport to 2.7" label? For most of the PRs I've reviewed I tend to avoid adding it myself, but I've seen it used periodically. It seems to be used rather infrequently ( https://github.com/python/cpython/pulls?q=is%3Apr+label%3A%22needs+backport+...), but I'm not entirely clear on when it should be used, particularly for documentation-related PRs.
It should be used where appropriate. ;) It's a per-case thing where you have to balance backwards-compatibility with whatever you're changing. Plus you have to have someone doing the PR review who cares enough to deal with the (good) chance there will be a merge conflict in the backport.