Block device general tests #8430
Conversation
It does not look like a target update but rather docs update and also adding tests |
| 1. [Hardware requirements](#hardware-requirements) | ||
| 2. [Usage](#usage) | ||
| - [Compile the test](#compile-the-test) | ||
| - [Run the test](#run-the-test) |
There was a problem hiding this comment.
Link doesnt work (I think text and link should be "run the example")
|
|
||
| ## Changing the block device | ||
|
|
||
| In Mbed OS, a C++ classes that inherits from the [BlockDevice](https://os.mbed.com/docs/latest/reference/storage.html#block-devices) |
There was a problem hiding this comment.
Typo: class instead of classes
|
@AnotherButler Please review the readme file |
|
@theamirocohen Curious. What was the reason for removing the PR type from the description? |
|
/morph build |
Build : FAILUREBuild number : 3413 |
|
@theamirocohen there are some build errors in the build, please take a look |
|
@theamirocohen It looks like this also needs to be rebased. Will manually restart the CI pipeline once the PR is updated, assuming GitHub is still experiencing issues. |
Copy edit file, mostly for active voice.
theamirocohen
force-pushed
the
block_device_general_tests
branch
from
e587d27 to
7962e0c
Compare
|
seems to be rebased now. |
Build : SUCCESSBuild number : 3437 Triggering tests/morph test |
Exporter Build : FAILUREBuild number : 3058 |
|
failure seems unrelated. rechecking |
Test : FAILUREBuild number : 3226 |
Exporter Build : SUCCESSBuild number : 3059 |
|
/morph build |
|
@theamirocohen Any more commits you'd like to add to the PR? |
|
Oh, I need to put a sock in my mouth. Didn't notice that commit fixed something... |
Build : SUCCESSBuild number : 3442 Triggering tests/morph test |
Exporter Build : SUCCESSBuild number : 3063 |
Test : SUCCESSBuild number : 3230 |
Suite of tests that can be easily configured to use a custom block device