Skip to content

Drivers: fix typos in doxygen #8564

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

Merged
merged 1 commit into from
Oct 31, 2018
Merged

Conversation

kegilbert
Copy link
Contributor

Description

Typo fixes in Doxygen for Drivers.

Pull request type

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

@cmonr
Copy link
Contributor

cmonr commented Oct 29, 2018

@kegilbert Anothre rebase, plx.

@AnotherButler When you get a chance to review.

@cmonr
Copy link
Contributor

cmonr commented Oct 30, 2018

@kegilbert Another "Anothre rebase, plx." plx

@kegilbert
Copy link
Contributor Author

@cmonr rebased

@cmonr
Copy link
Contributor

cmonr commented Oct 30, 2018

@kegilbert Rip. Docs failures.

@kegilbert kegilbert force-pushed the driver-doxy-typo-fix branch from 3532a15 to e8e9076 Compare October 30, 2018 20:05
@kegilbert
Copy link
Contributor Author

kegilbert commented Oct 30, 2018

Updated, missed a merge conflict grumble grumble. Are the doxy update fixes in so Travis is catching these now?

@cmonr
Copy link
Contributor

cmonr commented Oct 30, 2018

Updated, missed a merge conflict grumble grumble. Are the doxy update fixes in so Travis is catching these now?

#8593 will update the binary, but disable failure on the specific check. I don't have the bandwidth to learn and track down the doxygen issues atm, but the stdout is still enabled so that someone else can.

Once those issues are fixed, the updated command in that PR can be uncommented, enabled, and merged.

@kegilbert
Copy link
Contributor Author

Thanks, sounds good. I can try to take a look at the backlog of new Doxy warnings.

@cmonr
Copy link
Contributor

cmonr commented Oct 30, 2018

Thanks, sounds good. I can try to take a look at the backlog of new Doxy warnings.

🎉 🐦

@cmonr
Copy link
Contributor

cmonr commented Oct 31, 2018

/morph build

@mbed-ci
Copy link

mbed-ci commented Oct 31, 2018

Build : SUCCESS

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

Triggering tests

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

@mbed-ci
Copy link

mbed-ci commented Oct 31, 2018

@mbed-ci
Copy link

mbed-ci commented Oct 31, 2018

@0xc0170 0xc0170 changed the title Fix typos in Drivers doxygen Drivers: fix typos in doxygen Oct 31, 2018
@0xc0170 0xc0170 merged commit 8976709 into ARMmbed:master Oct 31, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants