msyavuz commented on PR #40439: URL: https://github.com/apache/superset/pull/40439#issuecomment-4564490733
So, i think the stacked pr's are a bit off now. All the pr's for the feature are opened against the feature branch, not on top of each other which means the last pr on the list have all the changes already. This is an antipattern in which you will have conflicts after each pr is merged and we can't parallelize the reviews. We should make each pr's base branch the previous pr instead so we avoid these. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
