-
Notifications
You must be signed in to change notification settings - Fork 3k
Add missing Doxygen descriptions for Nanostack #6313
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
Conversation
@SeppoTakalo Please review travis docs failure |
@SeppoTakalo Any update on the Travis doc failures? @AnotherButler Ping for review. |
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.
You may want @SenRamakri to confirm the Doxyfile options are right.
Fix couple of parameter issues as well.
Fixed couple of remaining Doxygen warnings from Mesh API. |
Done, added as a reviewer. /morph build |
Build : SUCCESSBuild number : 1513 Triggering tests/morph test |
Exporter Build : FAILUREBuild number : 1152 |
. . . |
Exporter Build : FAILUREBuild number : 1160 |
No obvious errors. Relaunching. |
Exporter Build : SUCCESSBuild number : 1164 |
@AnotherButler I did some changes after your last review, so the status is now lost. Please re-review. |
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.
LGTM 👍
Looks like tests are not triggered. re-triggering /morph test |
Test : FAILUREBuild number : 1355 |
|
@0xc0170 what is you're suggestion looks like unstable test from the console logs
Re-triggering test/morph test |
@ARMmbed/team-st-mcd Can you review this failure ? There was similar failures previously that were resolved |
Test : SUCCESSBuild number : 1357 |
Description