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: Update shell code fences for new website #15522

Merged

Conversation

olgashi
Copy link
Contributor

@olgashi olgashi commented Jan 14, 2022

Fixes #15518

Prerequisites checklist

What is the purpose of this pull request? (put an "X" next to an item)

[ ] Documentation update
[ ] Bug fix (template)
[ ] New rule (template)
[ ] Changes an existing rule (template)
[ ] Add autofix to a rule
[ ] Add a CLI option
[ ] Add something to the core
[x] Other, please explain:
Updating instances of shell code fences in /docs folder from old format ```sh
to new : ```shell

What changes did you make? (Give an overview)

Updated all instances of shell code fences in the docs/ folder to ```shell
The old format is not supported in the code formatter that is being used in the new website.
The ```shell is more universally supported, including on GitHub.

Is there anything you'd like reviewers to focus on?

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Jan 14, 2022

CLA Signed

The committers are authorized under a signed CLA.

@eslint-github-bot eslint-github-bot bot added triage feature labels Jan 14, 2022
nzakas
nzakas approved these changes Jan 15, 2022
Copy link
Member

@nzakas nzakas left a comment

Awesome! Thanks so much.

@nzakas nzakas merged commit 0469eb1 into eslint:main Jan 15, 2022
14 checks passed
@nzakas
Copy link
Member

nzakas commented Jan 27, 2022

@olgashi thanks so much for your work on this. We'd like to award you from our contributor pool. If you can email me at contact (at) eslint (dot) org, I can send you the details.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
archived due to age feature triage
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants