This repository was archived by the owner on Sep 23, 2023. It is now read-only.
Repository navigation
This repository was archived by the owner on Sep 23, 2023. It is now read-only.
Repeated Depends-On: merged again #81
Description
Follow-up to #32. Non-breaking, low-prio.
I've applied 2 patches, both Depends-On: the same patch. The latter was applied twice without causing any issues, but unnecessarily:
http://patchdemo.wmflabs.org/wikis/87ff0eb77076404c5541ac861a7eb158/w/
606306,17: checkboxHack: Trigger 'input' event when checkbox state changes
607404,1: [DNM][Patchdemo] Default to modern layout
606306,17: checkboxHack: Trigger 'input' event when checkbox state changes
606538,16: checkboxHack: Handle SPACE and ENTER key on button (label) to change checkbox state
In addition 606306,17 comes after 606538,16 in the chain, thus applying that can be eliminated too. As I see patches in chains are applied individually, thus keeping a set of already applied patches should be easy.
Activity
edg2s commentedon Feb 4, 2021
From #204:
I guess we are still applying the patch twice, but we aren’t showing it twice.