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

docs: user documentation on github pages #195

Merged
merged 2 commits into from
Nov 4, 2021
Merged

docs: user documentation on github pages #195

merged 2 commits into from
Nov 4, 2021

Conversation

yitsushi
Copy link
Contributor

@yitsushi yitsushi commented Nov 2, 2021

What this PR does / why we need it:

  • Documentation site is powered by Docusaurus.
  • gRPC documentation is generated with protoc-gen-doc.
  • Getting started documentation is the as the quick-start guide, but
    with separate files, so it's easye to navigate and digest the content.
    It really needs a refactoring and we have to write a real user Getting
    Started docoument.
  • Enabled blog for now, we can turn it off if we don't want to use it.
    Created an example blog post about the v0.1.0-alpha.1 release.
  • GitHub action to check if the documentation build process is not broken.
  • GitHub action to deploy documentation from main on push.

Tools:

Which issue(s) this PR fixes:
Fixes #85
Fixes #194

Special notes for your reviewer:

Notice: This is more like an initial documentation site.

Changed the workflow to run on this branch on push and it was able to deploy the documentation site: 61ef4be

Checklist:

  • squashed commits into logical changes
  • includes documentation

@yitsushi yitsushi added kind/documentation Documentation only kind/feature New feature or request labels Nov 2, 2021
@github-actions
Copy link
Contributor

github-actions bot commented Nov 2, 2021

This PR is so big! Please, split it 😊

@yitsushi yitsushi removed size/xl kind/feature New feature or request labels Nov 2, 2021
@codecov-commenter
Copy link

codecov-commenter commented Nov 2, 2021

Codecov Report

Merging #195 (3173bde) into main (fe9bb9f) will not change coverage.
The diff coverage is n/a.

❗ Current head 3173bde differs from pull request most recent head ed461d5. Consider uploading reports for the commit ed461d5 to get more accurate results
Impacted file tree graph

@@           Coverage Diff           @@
##             main     #195   +/-   ##
=======================================
  Coverage   55.81%   55.81%           
=======================================
  Files          44       44           
  Lines        2012     2012           
=======================================
  Hits         1123     1123           
  Misses        781      781           
  Partials      108      108           

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 f8622b6...ed461d5. Read the comment docs.

@yitsushi
Copy link
Contributor Author

yitsushi commented Nov 2, 2021

If you find any linguistic errors, if you have time for it, just commit to this branch. (around 97% of this is just copy-paste)

@github-actions
Copy link
Contributor

github-actions bot commented Nov 2, 2021

This PR is so big! Please, split it 😊

@github-actions
Copy link
Contributor

github-actions bot commented Nov 3, 2021

This PR is so big! Please, split it 😊

@github-actions
Copy link
Contributor

github-actions bot commented Nov 3, 2021

This PR is so big! Please, split it 😊

userdocs/docs/getting-started/basics/containerd.md Outdated Show resolved Hide resolved
@@ -0,0 +1,20 @@
# Dependencies
Copy link
Member

Choose a reason for hiding this comment

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

No entirely sure about using the top level userdocs/ directory. Personally, I think it should live under docs/ somewhere......but thats just a preference ;)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I tried to check how others are using docusaurus and I found the Profiles repo as a good candidate to see how they are using it and they have a userdocs in the root. Other (non-weaveworks) public repos are using a website top level directory.

Other than that, I like the separation of docs/ and the documentation website. This is not a document, it's an entire website and happens to use markdown, mostly, but can use japascript (node) and generate content. That would a bit weird to me if I open the docs directory and a see a lot of js files and markdown files with extra language elements in there (--- header, ::: blocks)

Copy link
Member

Choose a reason for hiding this comment

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

Yeah i take the point about separation.

This is not a document, it's an entire website

Then perhaps website or site is more appropriate then?

tags: [release, announcement]
---

# v0.1.0-alpha.1
Copy link
Member

Choose a reason for hiding this comment

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

I like that we can have a blog with this 👍

@github-actions
Copy link
Contributor

github-actions bot commented Nov 3, 2021

This PR is so big! Please, split it 😊

1 similar comment
@github-actions
Copy link
Contributor

github-actions bot commented Nov 3, 2021

This PR is so big! Please, split it 😊

@github-actions
Copy link
Contributor

github-actions bot commented Nov 3, 2021

This PR is so big! Please, split it 😊

userdocs/src/components/HomepageFeatures.js Outdated Show resolved Hide resolved
userdocs/docs/intro.md Outdated Show resolved Hide resolved
userdocs/docs/intro.md Outdated Show resolved Hide resolved
userdocs/docs/grpc/types/proto.md Outdated Show resolved Hide resolved
userdocs/docs/grpc/types/proto.md Show resolved Hide resolved

## Scalar Value Types

| .proto Type | Notes | C++ | Java | Python | Go | C# | PHP | Ruby |
Copy link
Member

Choose a reason for hiding this comment

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

this table does not render all too nicely, the 2nd from left column is very squished so you get 1 word per line

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Auto-generated with protobuf

Copy link
Member

Choose a reason for hiding this comment

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

hmmmm 😕

userdocs/docusaurus.config.js Outdated Show resolved Hide resolved
sidebar_position: 5
---

# Interacting with the service
Copy link
Member

Choose a reason for hiding this comment

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

we could add extra section here about simply building a client using the grpc methods

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That sounds more like a "write documentation" than "Initial creation of documentation site"

Copy link
Member

Choose a reason for hiding this comment

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

haha okay

@github-actions
Copy link
Contributor

github-actions bot commented Nov 3, 2021

This PR is so big! Please, split it 😊

3 similar comments
@github-actions
Copy link
Contributor

github-actions bot commented Nov 3, 2021

This PR is so big! Please, split it 😊

@github-actions
Copy link
Contributor

github-actions bot commented Nov 3, 2021

This PR is so big! Please, split it 😊

@github-actions
Copy link
Contributor

github-actions bot commented Nov 3, 2021

This PR is so big! Please, split it 😊

@Callisto13
Copy link
Member

chill out github-action bot!

@github-actions
Copy link
Contributor

github-actions bot commented Nov 3, 2021

This PR is so big! Please, split it 😊

yitsushi and others added 2 commits November 4, 2021 16:25
Notice: This is more like an initial documentation site

* Documentation site is powered by Docusaurus.
* gRPC documentation is generated with `protoc-gen-doc`.
* Getting started documentation is the as the quick-start guide, but
  with separate files, so it's easye to navigate and digest the content.
  It really needs a refactoring and we have to write a real user Getting
  Started docoument.
* Enabled `blog` for now, we can turn it off if we don't want to use it.
  Created an example blog post about the v0.1.0-alpha.1 release.
* CNAME: docs.flintlock.dev

Docusaurus: https://docusaurus.io/
protoc-gen-doc: https://github.com/pseudomuto/protoc-gen-doc

Fixes #85
* Apply suggestions from code review
* Update proto + readme based on feedback
* Remove ADR from the footer
* Minor doc changes

Co-authored-by: Claudia <claudiaberesford@gmail.com>
Co-authored-by: Richard Case <richard@weave.works>
Copy link
Member

@Callisto13 Callisto13 left a comment

Choose a reason for hiding this comment

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

happy with this as a starting point

@yitsushi yitsushi merged commit 1984d32 into main Nov 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use docs.flintlock.dev for documentation Initial creation of documentation site
5 participants