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

update TwoDTurb docs #36

Merged
merged 1 commit into from
Jan 3, 2020
Merged

update TwoDTurb docs #36

merged 1 commit into from
Jan 3, 2020

Conversation

dkarrasch
Copy link
Contributor

use unicode symbols

@codecov
Copy link

codecov bot commented Nov 27, 2019

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #36   +/-   ##
=======================================
  Coverage   98.64%   98.64%           
=======================================
  Files           6        6           
  Lines         518      518           
=======================================
  Hits          511      511           
  Misses          7        7

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 b15419e...a4d42fe. Read the comment docs.

@navidcy
Copy link
Member

navidcy commented Nov 28, 2019

thanks @dkarrasch
actually, the docs are bit outdated and we should re-think our strategy a bit on keeping them up to date.

@dkarrasch
Copy link
Contributor Author

One strategy could be to use https://github.com/JuliaDocs/DocStringExtensions.jl. Basically, you pepper your code with comments that would be helpful for code development anyway, and then build most of the documentation by macros. I.e., have docstrings build from the struct definitions, signatures of functions, fields of structs, etc. In the markdown file, one could have a few unspecific lines like "The package provides the following types/functions" and let Documenter fill in the details. Then you don't need to worry about the "hard-coded" text in the markdown, and only keep the docstrings up-to-date in the code.

@navidcy navidcy merged commit 204e04f into FourierFlows:master Jan 3, 2020
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.

2 participants