davids5 commented on pull request #1611: URL: https://github.com/apache/incubator-nuttx/pull/1611#issuecomment-691207971
> > I am fine with what @Ouss4 suggested. In that case, if the build blows up, do we revert the PR or just keep iterating until we get the master working building and working again? > > If it's an easy fix, we don't need to revert, we just provide another PR to address the issue. Yes that means we'll have a small window where things are broken, but that shouldn't stay for too long. If we can't see an immediate solution, we'll have to revert and look again at the problem at hands. @Ouss4 - thank you for outlining this. It seems workable. My latest practice for uptake-ing master I do this for both 'apps' and 'nuttx' - this ``` git fetch nuttx git checkout master-LAST git reset --hard nuttx/master git checkout master git pull master nuttx master ``` It allows me a quick exit if the master is broken. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
