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: Add style guide and more details for contributing. #519

Merged
merged 25 commits into from
Jul 12, 2023
Merged

Conversation

stoobie
Copy link
Collaborator

@stoobie stoobie commented Jun 6, 2023

Description of the Changes

This PR adds a style guide and more details to the contribution guide. Content is based on the OpenShift docs repo.

This content is only viewable in this repo, it is not published on docs.starknet.io or anywhere else.

PR Preview URL

There is no Antora preview for this content. Ignore the preview that is generated for this PR.

Check List

  • Changes have been done against dev branch, and PR does not conflict
  • PR title follows the convention: <docs/feat/fix/chore>(optional scope): <description>, e.g: fix: minor typos in code

This change is Reviewable

@stoobie stoobie requested a review from JameStark June 6, 2023 11:25
@github-actions
Copy link

github-actions bot commented Jun 6, 2023

Your preview build is ready! ✨ Check the following link in 1-2 minutes: https://starknet-io.github.io/starknet-docs/pr-519/documentation/ .

@stoobie stoobie marked this pull request as draft June 6, 2023 11:25
@stoobie stoobie changed the title feat: Add style guide and more details for contributing. WIP feat: Add style guide and more details for contributing. Jun 6, 2023
@github-actions
Copy link

github-actions bot commented Jun 6, 2023

Your preview build is ready! ✨ Check the following link in 1-2 minutes: https://starknet-io.github.io/starknet-docs/pr-519/documentation/ .

@github-actions
Copy link

github-actions bot commented Jun 6, 2023

Your preview build is ready! ✨ Check the following link in 1-2 minutes: https://starknet-io.github.io/starknet-docs/pr-519/documentation/ .

@github-actions
Copy link

github-actions bot commented Jun 6, 2023

Your preview build is ready! ✨ Check the following link in 1-2 minutes: https://starknet-io.github.io/starknet-docs/pr-519/documentation/ .

@github-actions
Copy link

github-actions bot commented Jun 7, 2023

Your preview build is ready! ✨ Check the following link in 1-2 minutes: https://starknet-io.github.io/starknet-docs/pr-519/documentation/ .

@JameStark JameStark force-pushed the dev branch 2 times, most recently from c8da654 to 0ba9303 Compare June 20, 2023 08:30
@github-actions
Copy link

github-actions bot commented Jul 3, 2023

Your preview build is ready! ✨ Check the following link in 1-2 minutes: https://starknet-io.github.io/starknet-docs/pr-519/documentation/ .

@github-actions
Copy link

github-actions bot commented Jul 3, 2023

Your preview build is ready! ✨ Check the following link in 1-2 minutes: https://starknet-io.github.io/starknet-docs/pr-519/documentation/ .

@github-actions
Copy link

github-actions bot commented Jul 4, 2023

Your preview build is ready! ✨ Check the following link in 1-2 minutes: https://starknet-io.github.io/starknet-docs/pr-519/documentation/ .

@github-actions
Copy link

Your preview build is ready! ✨ Check the following link in 1-2 minutes: https://starknet-io.github.io/starknet-docs/pr-519/documentation/ .

1 similar comment
@github-actions
Copy link

Your preview build is ready! ✨ Check the following link in 1-2 minutes: https://starknet-io.github.io/starknet-docs/pr-519/documentation/ .

@github-actions
Copy link

Your preview build is ready! ✨ Check the following link in 1-2 minutes: https://starknet-io.github.io/starknet-docs/pr-519/documentation/ .

@github-actions
Copy link

Your preview build is ready! ✨ Check the following link in 1-2 minutes: https://starknet-io.github.io/starknet-docs/pr-519/documentation/ .

@github-actions
Copy link

Your preview build is ready! ✨ Check the following link in 1-2 minutes: https://starknet-io.github.io/starknet-docs/pr-519/documentation/ .

@github-actions
Copy link

Your preview build is ready! ✨ Check the following link in 1-2 minutes: https://starknet-io.github.io/starknet-docs/pr-519/documentation/ .

@github-actions
Copy link

Your preview build is ready! ✨ Check the following link in 1-2 minutes: https://starknet-io.github.io/starknet-docs/pr-519/documentation/ .

@github-actions
Copy link

Your preview build is ready! ✨ Check the following link in 1-2 minutes: https://starknet-io.github.io/starknet-docs/pr-519/documentation/ .

@stoobie stoobie marked this pull request as ready for review July 10, 2023 15:05
@github-actions
Copy link

Your preview build is ready! ✨ Check the following link in 1-2 minutes: https://starknet-io.github.io/starknet-docs/pr-519/documentation/ .

@github-actions
Copy link

Your preview build is ready! ✨ Check the following link in 1-2 minutes: https://starknet-io.github.io/starknet-docs/pr-519/documentation/ .

@github-actions
Copy link

Your preview build is ready! ✨ Check the following link in 1-2 minutes: https://starknet-io.github.io/starknet-docs/pr-519/documentation/ .

@github-actions
Copy link

Your preview build is ready! ✨ Check the following link in 1-2 minutes: https://starknet-io.github.io/starknet-docs/pr-519/documentation/ .

Copy link
Contributor

@JameStark JameStark left a comment

Choose a reason for hiding this comment

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

Reviewable status: 0 of 7 files reviewed, 1 unresolved discussion (waiting on @stoobie)


contributing_to_docs/doc_guidelines.adoc line 29 at r1 (raw file):

. xref:starknet_docs_style_guide.adoc[_Starknet documentation style guide_] Reference this guide first. It provides guidance that is specific to Starknet documentation.
. link:https://redhat-documentation.github.io/supplementary-style-guide[_Red Hat supplementary style guide for product documentation_]. This guide overrides certain guidance from the _Google developer documentation style guide_.
. link:https://developers.google.com/style[_Google developer documentation style guide_]

This order differs from https://github.com/starknet-io/starknet-docs/blob/add_style_guide/contributing_to_docs/starknet_docs_style_guide.adoc which states that 'the primary source of style guidance for Starknet documentation.'

Code quote:

. link:https://redhat-documentation.github.io/supplementary-style-guide[_Red Hat supplementary style guide for product documentation_]. This guide overrides certain guidance from the _Google developer documentation style guide_.
. link:https://developers.google.com/style[_Google developer documentation style guide_]

Copy link
Contributor

@JameStark JameStark left a comment

Choose a reason for hiding this comment

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

Reviewable status: 0 of 7 files reviewed, 4 unresolved discussions (waiting on @stoobie)


README.adoc line 39 at r1 (raw file):

*What happens when you submit a PR?*

When you submit a PR, the https://github.com/orgs/openshift/teams/team-documentation[Starknet Docs team] reviews the PR and arranges further technical reviews as necessary. If the PR requires changes, the reviewers add comments in the PR. We might request that you make the changes, or let you know that we incorporated your content in a different PR. Occasionally, we might add commits to the original PR directly. When the PR has been reviewed and all updates are complete, the documentation team merges the PR and applies it to the valid version(s).

Should this link to Starknet?


contributing_to_docs/contributing.adoc line 20 at r1 (raw file):

*What happens when you submit a PR?*

When you submit a PR, the https://github.com/orgs/openshift/teams/team-documentation[Starknet Docs team] reviews the PR and arranges further technical reviews as necessary. If the PR requires changes, the reviewers add comments in the PR. We might request that you make the changes, or let you know that we incorporated your content in a different PR. Occasionally, we might add commits to the original PR directly. When the PR has been reviewed and all updates are complete, the documentation team merges the PR and applies it to the valid version(s).

Should this link to Starknet?


contributing_to_docs/doc_guidelines.adoc line 704 at r1 (raw file):

+
....
Create a secret that contains the certificate and key in the `openshift-config` namespace:

Starknet?

@stoobie stoobie requested a review from JameStark July 12, 2023 10:15
Copy link
Collaborator Author

@stoobie stoobie left a comment

Choose a reason for hiding this comment

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

Reviewable status: 0 of 7 files reviewed, 4 unresolved discussions (waiting on @JameStark)


README.adoc line 39 at r1 (raw file):

Previously, JameStark wrote…

Should this link to Starknet?

Done.


contributing_to_docs/doc_guidelines.adoc line 29 at r1 (raw file):

Previously, JameStark wrote…

This order differs from https://github.com/starknet-io/starknet-docs/blob/add_style_guide/contributing_to_docs/starknet_docs_style_guide.adoc which states that 'the primary source of style guidance for Starknet documentation.'

Well, this section talks about in what order ot use the guides. The section your quoting just names the guides. On that same page that you're quoting from, the next section provides the same order.


contributing_to_docs/doc_guidelines.adoc line 704 at r1 (raw file):

Previously, JameStark wrote…

Starknet?

Removed.


contributing_to_docs/contributing.adoc line 20 at r1 (raw file):

Previously, JameStark wrote…

Should this link to Starknet?

I deleted this file and moved any remaining content to doc_guidelines.adoc.

@github-actions
Copy link

Your preview build is ready! ✨ Check the following link in 1-2 minutes: https://starknet-io.github.io/starknet-docs/pr-519/documentation/ .

Copy link
Contributor

@JameStark JameStark left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewable status: 0 of 7 files reviewed, all discussions resolved

@github-actions
Copy link

Your preview build is ready! ✨ Check the following link in 1-2 minutes: https://starknet-io.github.io/starknet-docs/pr-519/documentation/ .

@stoobie stoobie changed the title WIP feat: Add style guide and more details for contributing. feat: Add style guide and more details for contributing. Jul 12, 2023
@stoobie stoobie merged commit a52bf4f into dev Jul 12, 2023
@stoobie stoobie deleted the add_style_guide branch July 12, 2023 11:05
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