Skip to content
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

README,doc: Add subdirectory READMEs and clean up documenation #170

Merged
merged 6 commits into from
May 19, 2020

Conversation

stotko
Copy link
Owner

@stotko stotko commented May 19, 2020

Although the documentation is linked at GitHub, these links are not easily discoverable in the README. Furthermore, users might look into doc/ to find the documentation which, however, will be a dead end. Add a dedicated section in the README to point at the documentation and create a local README in doc/ with the same section content. Furthermore, create a local README in examples/ as well to explain the structure and requirements of the examples. Finally, clean up the documentation style and remove unintuitive doxygen pages.

@codecov
Copy link

codecov bot commented May 19, 2020

Codecov Report

Merging #170 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #170   +/-   ##
=======================================
  Coverage   97.80%   97.80%           
=======================================
  Files          28       28           
  Lines        1825     1825           
=======================================
  Hits         1785     1785           
  Misses         40       40           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bb06b36...09cd323. Read the comment docs.

@stotko stotko merged commit 7b9afd9 into master May 19, 2020
@stotko stotko deleted the readme_doc_links branch May 19, 2020 16:16
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.

1 participant