linguini1 commented on PR #19041: URL: https://github.com/apache/nuttx/pull/19041#issuecomment-4632204681
> @linguini1 This is the base architecture changes and the commits are logically grouped. Splitting into multiple PR's isn't possible as the current scope is to have a working TriCore + SoC + Board. Each commit must pass the CI builds and not break anything as an atomic unit. So in theory if your commits follow that, each one could be a separate PR, which is much easier to review :) > otest and other basic test coverages are done. Which specific logs needs to be attached? Console or general test reports? Those logs are fine! I just wasn't able to find them quickly via your link. It's best if you attach them as files to the PR and/or copy paste them as text. Probably the former is best. -- 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]
