Skip to content

Rollup PR: UK Docathon pt2 #8552

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 78 commits into from
Oct 27, 2018
Merged

Rollup PR: UK Docathon pt2 #8552

merged 78 commits into from
Oct 27, 2018

Conversation

cmonr
Copy link
Contributor

@cmonr cmonr commented Oct 26, 2018 •
edited
Loading

Description

Rollup PR for the following PRs:

Added on Oct 26th:

Pull request type

[ ] Fix
[ ] Refactor
[ ] Target update
[ ] Functionality change
[x] Docs update
[ ] Test update
[ ] Breaking change

simonbutcher and others added 30 commits September 26, 2018 16:33
@simonbutcher
Update the Mbed TLS README.md
0fd0de6 
Add content missing from the README.md taken from the Yotta/Mbed OS 3 Readme.
@simonbutcher
Update Mbed TLS README.md followng review
fee476e 
Numerous changes to language, grammar, and corrections, following review.
@pan-
NonCopyable: Prevent doc generation for protected member function.
29f7d9d
@pan-
NonCopyable: Rewrite of class documentation.
3fe42b6
@pan-
NonCopyable: Add into platform doc group.
0d2a8e0
docs(api-condvar): Update documentation for the ConditionVariable API
c86ceb1 
- Hide protected member structures from doxygen.
- Add some more details/comments to the example.
- Add some defined/undefined behaviours.
- Add some additional details to function documentation
@paul-szczepanek-arm
spi doxygen fixes
482f2b6
@paul-szczepanek-arm
imperative
263f067
Copy edit NonCopyable.h
acaf968 
Make minor copy edits, mostly to existing text before this PR.
Copy edit ConditionVariable.h
8ddfdf5 
Copy edit file, including existing text.
@maciejbocianski
MbedCRC doxygen fix
394df12
@pan-
NonCopyable: Fix edit.
abeab68
@paul-szczepanek-arm
fix syntax err from misplaced ifdef
68628c1
@paul-szczepanek-arm
SPI capitalisation
d4b6fdc
@paul-szczepanek-arm
active voice
efdcdd5
@pan-
Doxygen: Move free standing HAL tests module inside their related HAL…
85bd15c 
… module.
@paul-szczepanek-arm
update internetsocket docs
fd0612b
@simonbutcher
Improve Mbed TLS README.md
650d8dc 
Improves the language, formatting and clarity of the Mbed TLS README.md.
PlatformMutex docs update
e095c78
@pan-
Doxygen: Hide friend declarations in dox.
836a6fc
@paul-szczepanek-arm
doxy cleanup
8a7d96c
@paul-szczepanek-arm
doxygen fixes
6cdda58
@paul-szczepanek-arm
return codes
5837e37
@paul-szczepanek-arm
remove redundancy and maintainance overhead
e31337d
@paul-szczepanek-arm
missing doxy
3929f56
@paul-szczepanek-arm
incorrect doc was copy pasted
4f2645b 
replaced with copydoc instead
Edit PlatformMutex.h
1131d84 
Change passive to active voice.
editorial changes, passive to active, removing redundancy
8c2bd40
@paul-szczepanek-arm
Update CellularBase.h
de261a4
@paul-szczepanek-arm
Update WiFiInterface.h
5241c3b
@mbed-ci
Copy link

mbed-ci commented Oct 26, 2018

Build : FAILURE

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

Breaking behaviors into two sections
0d128d1 
as per @kjbracey-arm review
@melwee01
Copy link
Contributor

OK, I've made what I think are my final changes.

@0xc0170
Copy link
Contributor

0xc0170 commented Oct 26, 2018

Condition variable looks to be the one having an error, and its changed in this rollup, we will need to reproduce locally. I suspect something with preprocessor as we had similar problem yesterday in SPI doxy changes

@cmonr
Copy link
Contributor Author

cmonr commented Oct 26, 2018

/morph build

This was referenced Oct 26, 2018
Merged
Merged
Merged
Merged
@mbed-ci
Copy link

mbed-ci commented Oct 26, 2018

Build : FAILURE

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

@cmonr
Copy link
Contributor Author

cmonr commented Oct 26, 2018

/morph build

@mbed-ci
Copy link

mbed-ci commented Oct 26, 2018

Build : SUCCESS

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

Triggering tests

/morph test
/morph export-build
/morph mbed2-build

@mbed-ci
Copy link

mbed-ci commented Oct 27, 2018

Test : SUCCESS

Build number : 3269
Test logs :http://mbed-os-logs.s3-website-us-west-1.amazonaws.com/?prefix=logs/8552/3269

@mbed-ci
Copy link

mbed-ci commented Oct 27, 2018

Exporter Build : SUCCESS

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

@cmonr cmonr merged commit d23d785 into ARMmbed:master Oct 27, 2018
@cmonr cmonr removed the needs: CI label Oct 27, 2018
@cmonr cmonr deleted the rollup branch October 30, 2018 00:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@0xc0170 0xc0170 0xc0170 approved these changes

@melwee01 melwee01 melwee01 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
10 participants
@cmonr @mbed-ci @0xc0170 @melwee01 @adbridge @simonbutcher @pan- @paul-szczepanek-arm @maciejbocianski @geky