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

feat(docs): add documentation website based on the Fermyon docs template #74

Merged
merged 4 commits into from
Jun 5, 2022

Conversation

radu-matei
Copy link
Member

@radu-matei radu-matei commented May 10, 2022

Big thanks to @jdbohrman for starting the work on this.
supersedes #66

This commit adds a documentation website in the docs/ directory for Bartholomew.
The website is based on the Fermyon docs website template, and is only focused on
the Bartholomew documentation, and not on showcasing all of Bartholomew's potential
features.
As such, not all pages and templates have been kept from the previous top-level
directories. Those should be part of the Bartholomew website template instead.

Signed-off-by: Radu Matei radu.matei@fermyon.com

Signed-off-by: Radu Matei <radu.matei@fermyon.com>
This commit adds a documentation website in the `docs/` directory for Bartholomew.
The website is based on the Fermyon docs website template, and is only focused on
the Bartholomew documentation, and not on showcasing all of Bartholomew's potential
features.
As such, not all pages and templates have been kept from the previous top-level
directories. Those should be part of the Bartholomew website template instead.

Signed-off-by: Radu Matei <radu.matei@fermyon.com>
Copy link
Member

@vdice vdice left a comment

Choose a reason for hiding this comment

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

Site runs great for me locally! Added a handful of small suggestions -- though, admittedly, many aren't from the changes in this PR, so they are optional. If not corrected in this PR, we should create a follow-up fix PR...

docs/content/index.md Outdated Show resolved Hide resolved
docs/content/templates.md Outdated Show resolved Hide resolved
docs/content/templates.md Outdated Show resolved Hide resolved
docs/content/scripting.md Outdated Show resolved Hide resolved
docs/content/markdown.md Show resolved Hide resolved
date = "2022-05-08T14:05:02.118466Z"

[extra]
url = "https://github.com/fermyon/bartholomew/blob/main/docs/content/contributing.md"
Copy link
Member

Choose a reason for hiding this comment

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

We may want to file an issue/reminder to populate this page.

Copy link
Member Author

Choose a reason for hiding this comment

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

I think we could construct this using https://github.com/fermyon/bartholomew/blob/main/docs/content as base?

Signed-off-by: Radu Matei <radu.matei@fermyon.com>
@radu-matei
Copy link
Member Author

I addressed all the comments, @vdice, I think this is ready for one final look.

Thanks!

Copy link
Member

@vdice vdice left a comment

Choose a reason for hiding this comment

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

One very minor outstanding typo (#74 (comment)); other than that, LGTM!

Signed-off-by: Radu Matei <radu.matei@fermyon.com>
@radu-matei radu-matei merged commit 7ecec5c into fermyon:main Jun 5, 2022
@radu-matei radu-matei deleted the website branch June 5, 2022 03:23
@radu-matei radu-matei mentioned this pull request Jul 12, 2022
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.

2 participants