Re: [PR] apps/testing:merge case folder to the new mm folder [nuttx-apps]

2025-01-23 Thread via GitHub
xiaoxiang781216 merged PR #2965: URL: https://github.com/apache/nuttx-apps/pull/2965 -- 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: commits-unsubscr...

Re: [PR] apps/testing:merge case folder to the new mm folder [nuttx-apps]

2025-01-22 Thread via GitHub
xiaoxiang781216 commented on PR #2965: URL: https://github.com/apache/nuttx-apps/pull/2965#issuecomment-2608976201 @txy-21 please update nuttx board's defconfig too -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the U

Re: [PR] apps/testing:merge case folder to the new mm folder [nuttx-apps]

2025-01-22 Thread via GitHub
xiaoxiang781216 commented on code in PR #2965: URL: https://github.com/apache/nuttx-apps/pull/2965#discussion_r1926335261 ## testing/mm/heaptest/Kconfig: ## @@ -3,32 +3,33 @@ # see the file kconfig-language.txt in the NuttX tools repository. # -config TESTING_MM +config TEST

Re: [PR] apps/testing:merge case folder to the new mm folder [nuttx-apps]

2025-01-21 Thread via GitHub
xiaoxiang781216 commented on code in PR #2965: URL: https://github.com/apache/nuttx-apps/pull/2965#discussion_r1924594436 ## testing/mm/open_memstream/Make.defs: ## @@ -1,5 +1,5 @@ -# apps/testing/open

Re: [PR] apps/testing:merge case folder to the new mm folder [nuttx-apps]

2025-01-21 Thread via GitHub
cederom commented on code in PR #2965: URL: https://github.com/apache/nuttx-apps/pull/2965#discussion_r1924639044 ## testing/mm/resmonitor/Make.defs: ## @@ -1,5 +1,5 @@ -# apps/testing/resmonitor/Make.

Re: [PR] apps/testing:merge case folder to the new mm folder [nuttx-apps]

2025-01-21 Thread via GitHub
txy-21 commented on code in PR #2965: URL: https://github.com/apache/nuttx-apps/pull/2965#discussion_r1924619224 ## testing/mm/mmtool/Kconfig: ## @@ -32,3 +32,4 @@ config TESTING_MM_POWEROFF depends on BOARDCTL_POWEROFF endif + Review Comment: ok -- This is an

Re: [PR] apps/testing:merge case folder to the new mm folder [nuttx-apps]

2025-01-21 Thread via GitHub
xiaoxiang781216 commented on code in PR #2965: URL: https://github.com/apache/nuttx-apps/pull/2965#discussion_r1924594436 ## testing/mm/open_memstream/Make.defs: ## @@ -1,5 +1,5 @@ -# apps/testing/open

Re: [PR] apps/testing:merge case folder to the new mm folder [nuttx-apps]

2025-01-21 Thread via GitHub
nuttxpr commented on PR #2965: URL: https://github.com/apache/nuttx-apps/pull/2965#issuecomment-2606104440 [**\[Experimental Bot, please feedback here\]**](https://github.com/search?q=repo%3Aapache%2Fnuttx+13552&type=issues) No, this PR description does not fully meet the Nutt

[PR] apps/testing:merge case folder to the new mm folder [nuttx-apps]

2025-01-21 Thread via GitHub
txy-21 opened a new pull request, #2965: URL: https://github.com/apache/nuttx-apps/pull/2965 ## Summary This is one step that merge test case as discussed in https://github.com/apache/nuttx-apps/pull/2931. Create mm folder and move folders to the new mm folder. ## Impact