Spelling fixes and general readability improvements for TEST_APPS documentation #8067
Conversation
Extend copy edits.
There was a problem hiding this comment.
@bridadan 👍 Thanks for the great copy edits. I've added some more on top of yours.
|
Your edits look great, thanks @AnotherButler! |
| You probably want to add the log folder to your .mbedignore file to prevent issues with build commands becoming too long over the time. | ||
| You can also find the results from the tests in the `mbed-os/log` folder. | ||
|
|
||
| You probably want to add the log folder to your `.mbedignore` file to prevent issues with build commands becoming too long over the time. |
There was a problem hiding this comment.
why build command touch even log files? Could it be ignored by default?
There was a problem hiding this comment.
@SeppoTakalo @OPpuolitaival Could y'all address this question?
There was a problem hiding this comment.
I believe the concern is that mbed-os/log adds another include directory to the already long list of include directories. As long as no subdirectories are added to mbed-os/log, I think it'd be safe to remove this section in the docs.
There was a problem hiding this comment.
If we know that its always mbed-os/log then I would be happy to add that into .mbedignore file, just to be safe.
|
/morph build |
Build : SUCCESSBuild number : 3080 Triggering tests/morph test |
Test : SUCCESSBuild number : 2869 |
Exporter Build : SUCCESSBuild number : 2691 |
Description
This PR is part of the 5.10 OOB process.
I've included some spelling fixes and general readability improvements to the TEST_APPS documentation. None of the changes are particularly technical, I was just making tweaks as I read the document.
Pull request type