I am oppose to modyfing source files to support the tool. I don't think we
should go that direction.Sent from Samsung tablet.
-------- Original message --------From: "Schock, Johannes - NIVUS GmbH"
<johannes.sch...@nivus.com> Date: 3/15/20 5:47 AM (GMT-06:00) To:
dev@nuttx.apache.org Subject: Re: Should we relax precheck a little bit?
Hello,I filed a pull request that introduces an idea of controlling the
behaviour of nxstyle from inside the file under
test.https://github.com/apache/incubator-nuttx/pull/569This would allow to
address some special requirements that are needed in cornercases (e.g.
whitelisting), and, by using the ignore feature, to switch of testing for
ranges of the file that are correct but nxstyle isn't happy about it (yet
unaddressed cornercases).This would give a contributor/commiter the possibility
to explicitly pass the PR through the style check, and perhaps file an nxstyle
issue.From time to time we could grep for the ignore command and remove it if
the issue had been resolved.Please share your thoughts.JohannesP.S.: if some
aren't happy with the xml-like syntax, we can change this for sure
