acassis commented on PR #19050: URL: https://github.com/apache/nuttx/pull/19050#issuecomment-4643473738
> Maybe this discussion should be had with the mailing list since it concerns one of the "inviolables", but my preferred solution would be: > > * Modifying nxstyle to support a `.nxstyleignore` file or something similar, scoped to a single directory > > * Add all of these specific symbols to a `.nxstyleignore` in the micro-ros directory > > * Add some updated documentation explaining how `.nxstyleignore` can only be used for third-party libraries with public-facing code where it is not possible to modify the symbols to conform to nxstyle > > > As bonus, we should move all the globally ignored symbols to `.nxstyleignore` files for other locations mentioned in the "ref" comments, but I don't think it's fair to ask that of you. > > However, we should see what other people think. Maybe I'm just crazy :) I like the idea of create a .nxstyleignore inside specific apps/ directories, this way we avoid create white list that could leak to other places where it should track issues. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
