-
Notifications
You must be signed in to change notification settings - Fork 571
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
apps/testing:merge case folder to the new mm folder #2965
Conversation
[Experimental Bot, please feedback here] No, this PR description does not fully meet the NuttX requirements. Here's why and how to fix it: Missing Information in Summary:
Missing Information in Impact:
Missing Information in Testing:
Example of an Improved PR Description:
... output showing successful test run from apps/testing/mm/memtester ...
... output showing successful test run from apps/testing/mm/memtester ...
By providing more detail and explicitly addressing all the points in the template, the PR will be much clearer and easier for reviewers to evaluate. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @txy-21 :-) Here goes my conditional go assuming all @xiaoxiang781216 remarks are adressed :-)
e39c35f
to
e1eabc3
Compare
e1eabc3
to
d5d3991
Compare
@txy-21 please update nuttx board's defconfig too |
1.rename original mm folder to heaptest and move it to mm folder 2.move the following folders into the new mm folder: cachetest, heaptest, iob, kasantest, memstress, memtester, ramtest, stressapptest Signed-off-by: tengshuangshuang <[email protected]>
d5d3991
to
04e954b
Compare
Summary
This is one step that merge test case as discussed in #2931.
Create mm folder and move folders to the new mm folder.
Impact
Only affect apps/testing. The changes are as follows:
1.rename original mm folder to mmtools and move it to the new mm folder.
2.move the following folders into the new mm folder:
cachetest, iob, kasantest, memstress, memtester, open-memstream, ramtest, stressapptest
Testing
CI test.
depends on apache/nuttx#15659