vrmay23 commented on PR #20394: URL: https://github.com/apache/nuttx/pull/20394#issuecomment-5894689596
> This kind of change needs a mailing list discussion first. > > It is also a massive change, and needs to be split into multiple PRs. Perhaps we could work on re-organizing individual sections in each PR (i.e. re-organize filesystem docs in one PR) to make sure each patch is still reviewable to ensure correctness. > > I am not necessarily opposed to the idea; I think organizing pages in a way that's consistent with the source tree is pretty intuitive, and we can still link to these pages from other top-level summaries (i.e. OS components can provide links to each of the source-tree organized pages it needs to). However, it's important to get community consensus on this. Could you please send the proposal (and a link to this PR) to the mailing list to get some comments/discussion going? > > Another note: > > The 50 pages mentioned to have new/re-written content must each be a patch of their own. I do not think it's productive to merge AI-generated documentation for 50 pages in one large patch, since AI is generally still not capable of fully understanding the codebase, and there needs to be some review from maintainers on each new doc to make sure it's correct and well-written. This should be separate from re-organizing the docs. See the related issue here with regards to missing doc pages, which we really wanted to resolve before the 13.0.0 release: #11081 Sorry. There is NO 50 new 'AI' documentation in this PR. Like I said, this is just reorganization and new 8 pages were created -- 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]
