parthitce commented on PR #19041: URL: https://github.com/apache/nuttx/pull/19041#issuecomment-6496704348
> @xiaoxiang781216 I think who submit the PR first should have priority, but in fact a bigger PR is more difficult to be reviewed and merged. @parthitce I think as a rule of thumb: instead of submitting all the changes in a single PR it is all better to divide in smaller PRs to make it easier to review and get merged. Understood and clear. Split is happening now in our end. Will have more split of PR's per scope and testing is on-going across three suppored compilers. But regarding the duplicated effort, see the list below (@zyfeier @jerpelea) - https://github.com/apache/nuttx/pull/20018 - https://github.com/apache/nuttx/pull/20016 - https://github.com/apache/nuttx/pull/19953 - https://github.com/apache/nuttx/pull/19836 - https://github.com/apache/nuttx/pull/19829 - https://github.com/apache/nuttx/pull/19783 - https://github.com/apache/nuttx/pull/19766 - https://github.com/apache/nuttx/pull/21110 Looking at the timeline, this PR is dated June and all the duplication started in August. And @xiaoxiang781216 is aware of this PR and why duplicate when we are actively responding & working with community? And why dismiss review https://github.com/apache/nuttx/pull/21088 and merge? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
