xiaoxiang781216 commented on PR #16319: URL: https://github.com/apache/nuttx/pull/16319#issuecomment-2876648498
> > @sumpfralle not all spell error in the touch files fixed: > > yes, that's what I described before: I grouped my fixes into different categories in order to allow an easier review (separated into "trivial" / "easy" / "complicated" / "needs a decision"). > > Thus, I do not fix the spelling mistakes "by file", but "by category". This causes the checks to fail. > > For future _code contributors_ it is certainly fine and fair to request the fixing of spelling issues alongside code contributions. But that is a different situation. > > Thus, I would suggest to accept the imperfect state of the spelling checks for this PR. > > If accepting the current state is not suitable, then I would suggest to revert the "enable CI checks" commit, until my spelling fixing PRs are integrated. OK. -- 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
