Skip to content
This repository has been archived by the owner on Sep 4, 2020. It is now read-only.

Reorganize project for documentation via gitbook #158

Merged
merged 3 commits into from
Jun 21, 2017
Merged

Reorganize project for documentation via gitbook #158

merged 3 commits into from
Jun 21, 2017

Conversation

michaeltlombardi
Copy link
Contributor

  • Replaced all restructured text files with markdown files, transitioning the markup between them to keep the content equivalent.
  • Added scaffolding, including project build script and accompanying psake tasks, for managing project documentation via gitbook.
  • Updated .gitignore for the new file types and output.
  • Removed extraneous image and media folder not currently in use.
  • Tasks for documentation auto-generate reference docs for functions and tests based on the captured information from those files.

Related Issues: #70, #104

+ Replaced all restructured text files with markdown files, transitioning the markup between them to keep the content equivalent.
+ Added scaffolding, including project build script and accompanying psake tasks, for managing project documentation via gitbook.
+ Updated `.gitignore` for the new file types and output.
+ Removed extraneous image and media folder not currently in use.
+ Tasks for documentation auto-generate reference docs for functions and tests based on the captured information from those files.

Related Issue: #70, #104
@michaeltlombardi
Copy link
Contributor Author

You can find a live preview of the generated docs here:

https://michaeltlombardi.github.io/Vester/

@michaeltlombardi michaeltlombardi changed the title Reorganize project for documentation via gitbook WIP: Reorganize project for documentation via gitbook Jun 16, 2017
+ The CHANGELOG used a `:heart:` instead of a unicode character; this only
works in GFM, and so did not translate to documentation. Updated the header
to use the unicode heart instead.
+ The CHANGELOG has numerous referenced issues but github does not link to
the filed issues natively. Additionally, the CHANGELOG display was broken
in GitBook due to a mis-parsing of the issues as headers. Here we turned
each issue into an explicit link to the upstream issue tracker and fixed
both problems at once.
@michaeltlombardi
Copy link
Contributor Author

Second commit fixed some problems I missed with the change log, those should be resolved now.

@michaeltlombardi michaeltlombardi changed the title WIP: Reorganize project for documentation via gitbook Reorganize project for documentation via gitbook Jun 16, 2017
+ Test reference docs used the test-file name for their first level header and file name
in documentation. This is problematic because it assumes that the test file name was
written to be easily and cleanly human-parsable. That is not the case for numerous
tests. Updating the psake task for the reference docs makes them easier to discover
and easier to understand.
@brianbunke brianbunke self-assigned this Jun 19, 2017
Copy link
Contributor

@brianbunke brianbunke left a comment

Choose a reason for hiding this comment

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

🔥, seriously! I'm going to play around with the build/psake files to understand exactly how the build process works here.

I imagine I'll merge this tonight or tomorrow, in case I think of any questions, and as a last opportunity for other interested parties to comment.

@michaeltlombardi
Copy link
Contributor Author

Awesome! Let me know what, if any, issues you run into.

A good follow-up issue to this would be a concept doc around the build file.

Loosely, the psake tasks define a way for the executor to install the needed powershell modules, install chocolatey, use chocolatey to install software, and install npm packages - that covers the software prereqs we can't assume will be on every machine.

The project-specific tasks are more around the compile task which scaffolds out all the required docs files, including reference docs for the tests. Some of that code could probably be re-used for #157 or could leverage Get-VesterTest instead of doing it the way it's implemented here.

@brianbunke
Copy link
Contributor

brianbunke commented Jun 21, 2017

The basics, outputting .md files into a new generate_docs directory during a build process (e.g. Appveyor):

# This will install PS modules, gather environment info, and then call psake.ps1
  # If you run this locally, Posh needs admin rights for Chocolatey/NodeJS/etc. installs
.\build.ps1 -TaskList Compile
# EDIT 2017/08/21 - See #169 for a slight update on this

The whole nine yards for any interested parties to investigate on their own; creates a PDF (lol awesome) in the project root, and also serves up the site at http://localhost:4000 for super sweet local preview action:

.\build.ps1 -TaskList Compile,GenerateSite,GeneratePdf,LivePreview

:shipit:

@brianbunke
Copy link
Contributor

Let it be known that on Tuesday, June 20th, 2017, technology won the day.

There will be more battles on the road ahead, and you may grow weary yet. But for tonight, we will revel in our victory, and reflect on the sacrifices of others so that we may fully appreciate this achievement.

@brianbunke brianbunke merged commit 9c2672e into WahlNetwork:master Jun 21, 2017
@michaeltlombardi michaeltlombardi deleted the doc-update branch June 21, 2017 16:24
@brianbunke brianbunke mentioned this pull request Jun 21, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants