Skip to content
This repository has been archived by the owner on May 1, 2024. It is now read-only.

Docs mop up #908

Merged
merged 4 commits into from May 9, 2017
Merged

Docs mop up #908

merged 4 commits into from May 9, 2017

Conversation

MichaelNorman
Copy link
Contributor

@MichaelNorman MichaelNorman commented May 8, 2017

Description of Change

Wrapped up summary-complete of docs, marking large swaths of internal stuff.

PR Checklist

  • Has tests (if omitted, state reason in description)
  • Rebased on top of master at time of PR
  • Changes adhere to coding standard
  • Consolidate commits as makes sense

@StephaneDelcroix
Copy link
Member

@MichaelNorman this doesn't pass the docs-validation step :/

@MichaelNorman
Copy link
Contributor Author

@kingces95 Can you oblige with the docs-validation step? Thank you!

Copy link
Contributor

@kingces95 kingces95 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I rebased and pushed mdoc changes after a build. If you've made changes to this branch since your last push then when you pull they'll be conflicts. Hopefully you haven't done that (forgot to tell you that whenever you ask me to run mdoc you shouldn't touch the branch till I'm done).

@MichaelNorman MichaelNorman merged commit d6adff2 into master May 9, 2017
@adrianknight89 adrianknight89 mentioned this pull request May 9, 2017
4 tasks
@rmarinho rmarinho deleted the docs-mop-up branch June 22, 2017 10:51
@samhouts samhouts added this to the 2.4.0 milestone Jul 3, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants