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

Resolve doxygen warning in filesystems docs #5395

Merged
merged 1 commit into from Nov 1, 2017

Conversation

Projects
None yet
6 participants
@kegilbert
Contributor

kegilbert commented Oct 27, 2017

Came up while writing docs for the filesystem APIs that we had class references for all blockdevice classes but ProfilingBlockDevice. @AnotherButler mentioned we had removed filesystems once from the doxygen exclusion list, so looks like we had all but that file generated before it was added back to the exclusion list.

Fixed the one warning we had in doxygen, @sg- was there a reason features/filesystem should be on the exclusion list other than the previous warnings?

@kegilbert kegilbert requested a review from sg- Oct 27, 2017

@sg-

This comment has been minimized.

Member

sg- commented Oct 28, 2017

Fixed the one warning we had in doxygen, @sg- was there a reason features/filesystem should be on the exclusion list other than the previous warnings?

Nope.

@sg-

sg- approved these changes Oct 28, 2017

@adbridge adbridge added needs: CI and removed needs: review labels Oct 30, 2017

@adbridge

This comment has been minimized.

Contributor

adbridge commented Oct 30, 2017

/morph build

@mbed-ci

This comment has been minimized.

mbed-ci commented Oct 30, 2017

Build : SUCCESS

Build number : 368
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/5395/

Triggering tests

/morph test
/morph uvisor-test

@mbed-ci

This comment has been minimized.

@mbed-ci

This comment has been minimized.

mbed-ci commented Oct 31, 2017

Build : SUCCESS

Build number : 392
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/5395/

Triggering tests

/morph test
/morph uvisor-test

@mbed-ci

This comment has been minimized.

@theotherjimmy theotherjimmy merged commit 9dfbf22 into ARMmbed:master Nov 1, 2017

5 checks passed

AWS-CI uVisor Build & Test Success
Details
ci-morph-build build completed
Details
ci-morph-test test completed
Details
continuous-integration/jenkins/pr-head This commit looks good
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@kegilbert kegilbert deleted the kegilbert:resolve-doxy-warnings-filesystem branch Nov 1, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment