LuchianMihai commented on PR #15794: URL: https://github.com/apache/nuttx/pull/15794#issuecomment-2651205383
> > > > Hi @acassis, On the same page, an paragraph before, talks about nxstyle, which can be used to do the same thing. Should we move that sentence to your new heading as well? > > > > > > > > > @LuchianMihai the script just uses the nxstyle, normally we don't use it directly (although it is possible) > > > > > > Hi @acassis, I do not want to delay this pull request. I was just pointing out that we already talk out the use of nxstyle on the page but we recommend using checkpatch. It may create confusion (mostly talking about me when I submitted an PR on nxstyle) about what is the relation between them. Your comment can be useful info for someone new, but nevertheless, do not delay the merge because of my comment > > @LuchianMihai maybe we should remove that reference to nxstyle, because I'm suggesting the standard way that most (if not all) developers do it. I think our documentation needs to be simple, clear and direct. That was not the case. Agree -- 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: commits-unsubscr...@nuttx.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
