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

Setup basic versioned docs #896

Merged
merged 12 commits into from Oct 24, 2019
Merged

Setup basic versioned docs #896

merged 12 commits into from Oct 24, 2019

Conversation

coilysiren
Copy link
Member

@coilysiren coilysiren commented Sep 16, 2019

Motivation

My proposal for v2 release cycle requires there be versioned docs (see #826). These was also a recent issue where people discussed versioned documentation (see #877).

Context and Future Work

This isn't "true" versioned documentation, not yet at least. In the future I would like to consider mkdocs, docsify, docusaurus or similar, as mentioned in this comment #877 (comment). This specific PR is focused only on setting up a v1 and v2 folder so that #892 won't have to deal with merge conflicts. Judge this PR by that parameter, eg, does this PR help make #892 easier?

Changes

  • creates a docs folder
  • moves non-versioned documentation (like the contributing guide) to the docs folder. the code of conduct has not been moved because (I think) github looks for it in the root directory (I'll explore this more in the future)
  • moves the versioned documentation into docs/v1/manual.md
  • adds an empty folder for v2, docs/v2/, with a .keep file (this pattern is commonly used with rails projects)
  • updates the tests for the new paths

@coilysiren coilysiren self-assigned this Sep 16, 2019
@coilysiren coilysiren changed the title Setup versioned docs Setup versioned docs (part 1) Sep 16, 2019
@coilysiren coilysiren changed the title Setup versioned docs (part 1) Setup basic versioned docs Sep 16, 2019
@codecov
Copy link

codecov bot commented Sep 16, 2019

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #896   +/-   ##
=======================================
  Coverage   71.64%   71.64%           
=======================================
  Files          30       30           
  Lines        2409     2409           
=======================================
  Hits         1726     1726           
  Misses        577      577           
  Partials      106      106

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 397c7d6...a16e68e. Read the comment docs.

@coilysiren coilysiren marked this pull request as ready for review September 16, 2019 05:38
@coilysiren coilysiren requested a review from a team as a code owner September 16, 2019 05:38
@coilysiren coilysiren mentioned this pull request Sep 18, 2019
@coilysiren
Copy link
Member Author

🕙 PR timeout!

@coilysiren coilysiren closed this Oct 2, 2019
@AudriusButkevicius AudriusButkevicius deleted the versioned-docs branch October 2, 2019 07:50
@coilysiren coilysiren restored the versioned-docs branch October 12, 2019 07:52
@coilysiren coilysiren reopened this Oct 12, 2019
Copy link
Member

@saschagrunert saschagrunert left a comment

Choose a reason for hiding this comment

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

I like that 👍 🙂

@asahasrabuddhe
Copy link
Member

I have a suggestion. How about we use good old godoc for the library? That would be consistent and would allow for auto-generated and would allow for playground-style runnable examples.

We can always generate a README.md from the godoc.

Example link: https://go.ajitem.com/pkg/go.ajitem.com/stack/

@coilysiren
Copy link
Member Author

@asahasrabuddhe can you create an issue about godoc? This PR isn't quite the place to have the conversation.

@coilysiren coilysiren merged commit 850de85 into master Oct 24, 2019
@coilysiren coilysiren deleted the versioned-docs branch November 21, 2019 09:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants