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

Generate stub documentation #66

Merged
merged 3 commits into from
May 28, 2016
Merged

Generate stub documentation #66

merged 3 commits into from
May 28, 2016

Conversation

mortenpi
Copy link
Member

Addresses #57.

Needs more polish before it can be merged, but thoughts on the exact API would be welcome. Right now I propose a very simple Documenter.generate(pkgname).

Also, no format argument at the moment, just defaulting to mkdocs.

@MichaelHatherly
Copy link
Member

Also, no format argument at the moment, just defaulting to mkdocs.

Sounds good for the moment. We can add to it later once #3 and #4 get implemented.

but thoughts on the exact API would be welcome.

Looks good to me as is really.

Will need some kind of tests if possible, probably create a temp dir, run generate on it and check the structure looks alright.

@@ -8,6 +8,9 @@ Two functions are exported from this module for public use:
- [`makedocs`](@ref). Generates documentation from docstrings and templated markdown files.
- [`deploydocs`](@ref). Deploys generated documentation from *Travis-CI* to *GitHub Pages*.

Additionally it provides the unexported [`generate`](@ref), which can be used to
Copy link
Member

Choose a reason for hiding this comment

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

Perhaps [Documenter.generate](@ref) to make it more obvious that it's unexported?

@MichaelHatherly
Copy link
Member

MichaelHatherly commented May 26, 2016

Appveyor errors is unrelated. fixed now

"""
generate(
pkgname;
dir = "<package directory>"
Copy link
Contributor

Choose a reason for hiding this comment

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

Should read "<package directory>/docs" ?

@codecov-io
Copy link

codecov-io commented May 28, 2016

Current coverage is 79.76%

Merging #66 into master will decrease coverage by 5.19%

@@             master        #66   diff @@
==========================================
  Files            12         13     +1   
  Lines           804        840    +36   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
- Hits            683        670    -13   
- Misses          121        170    +49   
  Partials          0          0          

Powered by Codecov. Last updated by b1938cb...29bfa30

@mortenpi mortenpi changed the title WIP: Generate stub documentation Generate stub documentation May 28, 2016
@@ -88,6 +88,16 @@ Formats.Format
Formats.mimetype
```

## Generator
Copy link
Member

Choose a reason for hiding this comment

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

Blank line after headers, just to be consistent with the rest of the package.

@MichaelHatherly
Copy link
Member

Other than a couple of tiny nitpicks, LGTM.

"""
function index(pkgname)
"""
# $(pkgname).jl
Copy link
Member

Choose a reason for hiding this comment

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

As with the other comment, blank line after header here too.

@mortenpi
Copy link
Member Author

Fixed the nitpicks.

@MichaelHatherly
Copy link
Member

Great, I'll merge once appveyor finishes. Then later this evening I'll move the package over to JuliaDocs (issue #69) and add you as a co-owner so you can have a bit more freedom to implement what you've got planned. Probably best for neither of us to push direct to master from that point on of course.

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.

4 participants