Alan C. Assis
Alan C. Assis
@anchao please include a commit log message with more info about this modification
@anchao what to do with this PR? Did you address Xiang Xiao comments?
@zdebanos I saw it is failing because of typos only: /home/runner/work/nuttx/nuttx/nuttx/arch/arm/src/stm32/Kconfig:10762: retransfered ==> retransferred /home/runner/work/nuttx/nuttx/nuttx/arch/arm/src/stm32/Kconfig:10765: transfered ==> transferred /home/runner/work/nuttx/nuttx/nuttx/arch/arm/src/stm32/Kconfig:10766: reseted ==> reset /home/runner/work/nuttx/nuttx/nuttx/arch/arm/src/stm32/stm32_i2cslave_v2.c:100: deffering ==> differing, deferring Please send a...
Yes, it is confuse, I think it should just show a summary with errors instead of the verbose log
Hi @TimJTi this print is from AVR PR: https://github.com/apache/nuttx/pull/16365
also on nau7802 driver: ``` struct i2c_msg_s readcmd[2] = { ```
@TimJTi good question! I think it used to work in the past, because Greg implemented nxstyle to be consistent with the Coding Style guy, but we need to confirm it,...
@TimJTi I downloaded NuttX 9.1 and noticed it also wasn't able to detect the issue, you are right, it is "FUBAR" ``` alan@dev:/tmp/patacongo-nuttx-e4e4cce69624/tools$ ./nxstyle /tmp/nau7802.c alan@dev:/tmp/patacongo-nuttx-e4e4cce69624/tools$ ``` It needs to...
@TimJTi strange, for some reason it didn't detect the issue in the CI. @lupyuen do you know why CI's nxstyle didn't catch this issue?
Thanks for confirming @simbit18 !