acassis commented on PR #18368: URL: https://github.com/apache/nuttx/pull/18368#issuecomment-3884790911
> > > Folks, for the future, documentation and board changes should be on a different commit. Other than that, great work! > > > > > > @fdcavalcanti true, but a problem of separating drivers, docs and drivers in different PRs is when we need to revert it some commit could be missed, specially if the PRs are merger with some intermediated PRs. > > A better approach is separating each logic part in commits inside the same PR. > > That was exactly my suggestion. Oops, sorry I read fast and thought PRs :-D :+1: -- 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]
