TAOS-CI icon indicating copy to clipboard operation
TAOS-CI copied to clipboard

Regard of 1PR/1Commit rule

Open DonghwanJeong opened this issue 4 years ago • 4 comments

Related of https://github.com/nnsuite/TAOS-CI/pull/596

1PR/1Commit rule may make interfere with software configuration management. It want to be reviewed using this system regardless commit count at same PR. Already branch merge rule is used at many project. For example, follow under git tree, this commit is refer by linux kernel.

* 872e24d - (HEAD -> master) hexagon: remove asm/bitsperlong.h (3 weeks ago) <Masahiro Yamada>
* 31f4f5b - (tag: v5.4-rc7) Linux 5.4-rc7 (3 weeks ago) <Linus Torvalds>
*   4486695 - Merge tag 'armsoc-fixes' of git://git.kernel.org/pub/scm/linux/kernel/git/soc/soc (3 weeks ago) <Linus Torvalds>
|\  
| * 002d3c6 - MAINTAINERS: update Cavium ThunderX2 maintainers (4 weeks ago) <Jayachandran C>
| *   e1af5a1 - Merge tag 'stm32-dt-for-v5.4-fixes-2' of git://git.kernel.org/pub/scm/linux/kernel/git/atorgue/stm32 into arm/fixes (4 weeks ago) <Olof Johansson>
| |\  
| | * f4d6e0f - ARM: dts: stm32: change joystick pinctrl definition on stm32mp157c-ev1 (4 weeks ago) <Amelie Delaunay>
| | * afe3af8 - ARM: dts: stm32: remove OV5640 pinctrl definition on stm32mp157c-ev1 (4 weeks ago) <Amelie Delaunay>
| | * 9df50c2 - ARM: dts: stm32: Fix CAN RAM mapping on stm32mp157c (4 weeks ago) <Christophe Roullier>
| | * 832c436 - ARM: dts: stm32: relax qspi pins slew-rate for stm32mp157 (4 weeks ago) <Patrice Chotard>
| * |   3a82cd4 - Merge tag 'imx-fixes-5.4-3' of git://git.kernel.org/pub/scm/linux/kernel/git/shawnguo/linux into arm/fixes (4 weeks ago) <Olof Johansson>
| |\ \  
| | * | f852497 - arm64: dts: zii-ultra: fix ARM regulator GPIO handle (4 weeks ago) <Lucas Stach>
| * | |   ba7f8c9 - Merge tag 'sunxi-fixes-for-5.4-2' of https://git.kernel.org/pub/scm/linux/kernel/git/sunxi/linux into arm/fixes (4 weeks ago) <Olof Johansson>
| |\ \ \  
| | * | | e690053 - ARM: sunxi: Fix CPU powerdown on A83T (5 weeks ago) <Ondrej Jirman>
| | * | | e614f34 - ARM: dts: sun8i-a83t-tbs-a711: Fix WiFi resume from suspend (5 weeks ago) <Ondrej Jirman>
| * | | |   70a7274 - Merge tag 'imx-fixes-5.4-2' of git://git.kernel.org/pub/scm/linux/kernel/git/shawnguo/linux into arm/fixes (4 weeks ago) <Olof Johansson>
| |\ \ \ \  
| | | |/ /  
| | |/| |   
| | * | | 958c601 - arm64: dts: imx8mn: fix compatible string for sdma (5 weeks ago) <Shengjiu Wang>
| | * | | e346ff9 - arm64: dts: imx8mm: fix compatible string for sdma (5 weeks ago) <Shengjiu Wang>

DonghwanJeong avatar Dec 03 '19 06:12 DonghwanJeong

:octocat: cibot: Thank you for posting issue #597. The person in charge will reply soon.

taos-ci avatar Dec 03 '19 06:12 taos-ci

I think you can disable that rule (plugin) if you want.

Such heavy branching as in Linux kernel is appropriate ONLY if we can afford to have a dedicated maintainer for each branch. In most projects, we can't.

myungjoo avatar Dec 03 '19 06:12 myungjoo

I added this issue into the Kanban board of TAOS-CI to handle 1PR/Ncommits (TODO) as well as 1PR/1commit (currently).

leemgs avatar Dec 03 '19 10:12 leemgs

I have added it to the official Kanban board to support the request positively.

  • 1PR / 1commit policy
  • 1PR / Ncommits policy (Coming soon in the near future)

leemgs avatar Dec 03 '19 10:12 leemgs