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.

Sign up for GitHub

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

Jump to bottom

dev_guide: Add Pygments Lexer examples #71458

Merged
merged 2 commits into from
Sep 24, 2020
Merged

dev_guide: Add Pygments Lexer examples #71458

merged 2 commits into from
Sep 24, 2020

Conversation

Akasurde
Copy link
Member

SUMMARY

Fixes: #51307

Signed-off-by: Abhijeet Kasurde akasurde@redhat.com

ISSUE TYPE
  • Docs Pull Request
COMPONENT NAME

docs/docsite/rst/dev_guide/style_guide/index.rst

@Akasurde
dev_guide: Add Pygments Lexer examples
262774f 
Fixes: ansible#51307

Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
@Akasurde Akasurde added this to To do in OLD Ansible Documentation via automation Aug 26, 2020
@Akasurde Akasurde moved this from To do to In progress in OLD Ansible Documentation Aug 26, 2020
@Akasurde Akasurde mentioned this pull request Aug 26, 2020
Closed
@ansibot ansibot added affects_2.11 core_review In order to be merged, this PR must follow the core review workflow. docs This issue/PR relates to or includes documentation. docsite This issue/PR relates to the documentation website. has_issue needs_triage Needs a first human triage before being processed. support:core This issue/PR relates to code supported by the Ansible Engineering Team. labels Aug 26, 2020
@acozine acozine removed the needs_triage Needs a first human triage before being processed. label Aug 26, 2020
@acozine acozine removed this from In progress in OLD Ansible Documentation Aug 26, 2020
@ansibot ansibot added the stale_ci This PR has been tested by CI more than one week ago. Close and re-open this PR to get it retested. label Sep 3, 2020
@acozine
Copy link
Contributor

acozine commented Sep 10, 2020

Thanks @Akasurde for the PR. I looked at existing examples in the Ansible docs, and we use a slightly different list of lexers:

yaml bash shell-session python powershell text ini json yaml+jinja shell csharp rst console sh ansible-output diff

@acozine acozine added the backport_request This issue has been fixed in devel and is a candidate for a backport. label Sep 24, 2020
@acozine acozine merged commit 4f990e7 into ansible:devel Sep 24, 2020
heiderich pushed a commit to heiderich/ansible that referenced this pull request Oct 4, 2020
@Akasurde @acozine @heiderich
dev_guide: Add Pygments Lexer examples (ansible#71458)
03c4ef1 
* dev_guide: Add Pygments Lexer examples

Fixes: ansible#51307

Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>

* adds ansible-output lexer, remmoves unsupported lexers

Co-authored-by: Alicia Cozine <acozine@users.noreply.github.com>
@ansible ansible locked and limited conversation to collaborators Oct 22, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@acozine acozine Awaiting requested review from acozine

@samccann samccann Awaiting requested review from samccann

Assignees
No one assigned
Labels
affects_2.11 backport_request This issue has been fixed in devel and is a candidate for a backport. core_review In order to be merged, this PR must follow the core review workflow. docs This issue/PR relates to or includes documentation. docsite This issue/PR relates to the documentation website. has_issue stale_ci This PR has been tested by CI more than one week ago. Close and re-open this PR to get it retested. support:core This issue/PR relates to code supported by the Ansible Engineering Team.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Document available lexers
3 participants
@Akasurde @acozine @ansibot