patacongo commented on pull request #1562: URL: https://github.com/apache/incubator-nuttx/pull/1562#issuecomment-677851809
I personally believe that the ARM change is independent and we should NOT hold up merging the change because additional functionality has been unfairly dumped onto the contributor. That is wrong. We should not do that. This encourages the contributor to cut corners and propose shitty modifications. And the submitter cannot test these changes ... we should not let untested changes into there repository either. I think you are treating the contributor unfairly. Let him merge the good, tested. ARM changes without forcing him to do a bunch of blind, shit changes to other architectures that he is unfamiliar with. That is stupid. ---------------------------------------------------------------- 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: us...@infra.apache.org