The Wayback Machine - https://web.archive.org/web/20201009213124/https://github.com/github/docs/pull/271
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

Fix indentation inside Code Examples #271

Open
wants to merge 1 commit into
base: main
from
Open

Conversation

@dtinth
Copy link

@dtinth dtinth commented Oct 8, 2020

Why:

In the Code Examples section, the YAML shown looks like, but is not, a valid GitHub Actions workflow file. The schedule: section is inside the on: where the docs shows it as a separate block.

What's being changed:

The indentation has been fixed to make the code example a valid workflow.

This helps the reader understand the context better that schedule: is not a standalone block, but is supposed to be in the on: section. It also prevents attentive readers from getting distracted by invalid YAML syntax in the code block while reading the guidelines.

Check off the following:

In the Code Examples section, the YAML shown looks like, but is not, a valid GitHub Actions workflow file.

The `schedule:` section is inside the `on:`.

This helps the reader understand the context better that `schedule:` is not a standalone block, but is supposed to be in the `on:` section.
@welcome
Copy link

@welcome welcome bot commented Oct 8, 2020

Thanks for opening this pull request! A GitHub docs team member should be by to give feedback soon. In the meantime, please check out the contributing guidelines.

@github-actions github-actions bot added this to Triage in Docs team reviews Oct 8, 2020
@janiceilene
Copy link
Contributor

@janiceilene janiceilene commented Oct 8, 2020

Thanks for opening a pull request @dtinth! I'll let the @github/docs-content-ecosystem team know this is ready for review 🌟

@janiceilene janiceilene moved this from Triage to Ecosystem in Docs team reviews Oct 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.
Morty Proxy This is a proxified and sanitized view of the page, visit original site.