-
Notifications
You must be signed in to change notification settings - Fork 249
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
Rules documentation #2932
Rules documentation #2932
Conversation
Thanks very much for this contribution @clbarrell !! At least one of these build errors in this PR is occurring because there's no entry for this page in the @clbarrell , whereabouts were you thinking of adding this page to in the nav? Is it meant to be part of:
There is some description of this in the Working with the docs site section of the Markdown syntax style guide. This content is going to be pulled out into its own 'public contributor's guide' page. |
Sorry @clbarrell , I didn't acknowledge the 'draft' label you added to this PR. I've converted this PR to a draft using the 'convert to draft' feature at the top-right (under Reviewers). Please click Ready for review when it's ready to review 😁 |
Preview URL: https://2932--bk-docs-preview.netlify.app |
b2d08eb
to
700dda9
Compare
Co-authored-by: L Suzuki <[email protected]>
Co-authored-by: L Suzuki <[email protected]>
Co-authored-by: L Suzuki <[email protected]>
Co-authored-by: L Suzuki <[email protected]>
Co-authored-by: L Suzuki <[email protected]>
Co-authored-by: L Suzuki <[email protected]>
Co-authored-by: L Suzuki <[email protected]>
Initially Rules only supports pipelines, but in the future it could be extended to support other resources (other Buildkite products). For now I think it makes sense to live within the Pipelines product docs, but can see it being promoted into the Platform section of the docs when we introduce non-pipelines rules, it's a platform capability but starting out scoped to only pipelines. |
Thanks for clarifying this! Therefore, I feel it makes sense that for the time being, we keep this as a top-level nav section in the Pipelines since moving it within the Configure pipelines section might imply a more narrowed scope (i.e. just for use within pipelines only, as opposed to use with other features of Buildkite Pipelines as a product). Edit: In retrospect, I should've realised clusters transcends pipelines, which adds more weight to its current location. |
… URL path, and tweak intro and 'create a rule' section of 'manage rules' page.
Just implemented some updates to the PR, particularly on the 'managing rules' page. These involve the following points:
|
…error in listing orgs.
<tbody> | ||
<tr> | ||
<th><code>type</code></th> | ||
<td>The rule type. Must match one of the <a href="/docs/pipelines/rules#rule-types">available rule types</a>.<br> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FYI only (for future docs contributions) ... Markdown syntax does not render into HTML, when this syntax is used within HTML tables in a Markdown file.
Therefore, I had to convert this Markdown link into its HTML equivalent.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ready to merge!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Adding one more tweak.
Starting a new Rules page for Matt and Laura.