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

Add docs about contributing to docs #46481

Merged
merged 19 commits into from
Oct 24, 2018
Merged

Conversation

acozine
Copy link
Contributor

@acozine acozine commented Oct 4, 2018

SUMMARY

Offers guidance for community members who want to contribute to the documentation.

Based on the AnsibleFest talk from 2018.

ISSUE TYPE
  • Docs Pull Request
COMPONENT NAME

docs.ansible.com

ANSIBLE VERSION

2.8

@ansibot
Copy link
Contributor

ansibot commented Oct 4, 2018

The test ansible-test sanity --test no-smart-quotes [explain] failed with 1 error:

docs/docsite/rst/community/documentation_contributions.rst:118:39: use ASCII quotes `'` and `"` instead of Unicode quotes

The test ansible-test sanity --test rstcheck [explain] failed with 2 errors:

docs/docsite/rst/community/documentation_contributions.rst:93:0: (bash) `make htmlsingle rst=<relative/path/to/your_file.rst>'
docs/docsite/rst/community/documentation_contributions.rst:93:0: (bash) syntax error near unexpected token `newline'

click here for bot help

@ansibot ansibot added WIP This issue/PR is a work in progress. Nevertheless it was shared for getting input from peers. affects_2.8 This issue/PR affects Ansible v2.8 ci_verified Changes made in this PR are causing tests to fail. docs This issue/PR relates to or includes documentation. 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 Oct 4, 2018
@samdoran samdoran removed the needs_triage Needs a first human triage before being processed. label Oct 4, 2018

Improving the documentation is an easy way to make your first contribution to the Ansible project. You don't have to be a programmer, since our documentation is written in YAML (module documentation) or reStructured Text. If you're using Ansible, you know YAML - playbooks are written in YAML. And rST is mostly just text. You don't even need git experience, if you use the ``Edit on GitHub`` option.

If you find a typo, a broken example, a missing topic, or any other error or omission on docs.ansible.com, let us know. Here are some ways to support Ansible documentation:
Copy link
Contributor

Choose a reason for hiding this comment

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

Could it be a link?

Joining the documentation working group
=======================================

The Documentation Working Group is just getting started, please visit the community repo for more information.
Copy link
Contributor

Choose a reason for hiding this comment

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

Where is located this community repo?

@ansibot ansibot removed the ci_verified Changes made in this PR are causing tests to fail. label Oct 9, 2018
@acozine acozine removed the WIP This issue/PR is a work in progress. Nevertheless it was shared for getting input from peers. label Oct 18, 2018
@acozine
Copy link
Contributor Author

acozine commented Oct 18, 2018

Ready for review. The branch is published (temporarily) at http://docs.testing.ansible.com/ansible/latest/community/documentation_contributions.html#community-documentation-contributions and associated pages.

@ansibot ansibot added the WIP This issue/PR is a work in progress. Nevertheless it was shared for getting input from peers. label Oct 18, 2018
@acozine acozine changed the title WIP: Contribute to docs Add docs about contributing to docs Oct 18, 2018
@acozine acozine removed the WIP This issue/PR is a work in progress. Nevertheless it was shared for getting input from peers. label Oct 18, 2018
@acozine acozine added this to To do in OLD Ansible Documentation via automation Oct 18, 2018
@ansibot ansibot added the core_review In order to be merged, this PR must follow the core review workflow. label Oct 18, 2018

Ansible has a lot of documentation and a small team of writers. Community support helps us keep up with new features, fixes, and changes.

Improving the documentation is an easy way to make your first contribution to the Ansible project. You don't have to be a programmer, since our documentation is written in YAML (module documentation) or reStructured Text. If you're using Ansible, you know YAML - playbooks are written in YAML. And rST is mostly just text. You don't even need git experience, if you use the ``Edit on GitHub`` option.
Copy link
Contributor

Choose a reason for hiding this comment

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

Shouldn't it be YAML-playbooks or YAML playbooks, and not YAML - playbooks?

Copy link
Contributor

Choose a reason for hiding this comment

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

or reStructured Text
or reStructured Text (RST)


``make htmlsingle`` adds ``rst/`` to the beginning of the path you provide in ``rst=``, so you can't type the filename with autocomplete. If you run ``make htmlsingle`` from the ``docs/docsite/rst/`` directory, you'll get ``make: *** No rule to make target `htmlsingle'. Stop.`` If you run it from ``docs/docsite/`` and provide the full path, you'll get ``sphinx-build: error: cannot find files ['rst/rst/community/documentation_contributions.rst']``.

2. Building an rST file with sphinx-build:
Copy link
Contributor

Choose a reason for hiding this comment

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

I wonder if you could his should be marked as advanced? I've never use it.

Maybe move to the end of the doc under and advanced section


Ansible has a lot of documentation and a small team of writers. Community support helps us keep up with new features, fixes, and changes.

Improving the documentation is an easy way to make your first contribution to the Ansible project. You don't have to be a programmer, since our documentation is written in YAML (module documentation) or reStructured Text. If you're using Ansible, you know YAML - playbooks are written in YAML. And rST is mostly just text. You don't even need git experience, if you use the ``Edit on GitHub`` option.
Copy link
Contributor

Choose a reason for hiding this comment

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

or reStructured Text
or reStructured Text (RST)

@@ -0,0 +1,166 @@
.. _community_documentation_contributions:
Copy link
Contributor

Choose a reason for hiding this comment

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

Should Module DOCUMENTATION in dev_guide link to this?


.. note::

``make htmlsingle`` adds ``rst/`` to the beginning of the path you provide in ``rst=``, so you can't type the filename with autocomplete. If you run ``make htmlsingle`` from the ``docs/docsite/rst/`` directory, you'll get ``make: *** No rule to make target `htmlsingle'. Stop.`` If you run it from ``docs/docsite/`` and provide the full path, you'll get ``sphinx-build: error: cannot find files ['rst/rst/community/documentation_contributions.rst']``.
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we break this into steps?

@ansibot
Copy link
Contributor

ansibot commented Oct 23, 2018

The test ansible-test sanity --test docs-build [explain] failed with 1 error:

docs/docsite/rst/community/documentation_contributions.rst:120:0: warning: Bullet list ends without a blank line; unexpected unindent.

The test ansible-test sanity --test rstcheck [explain] failed with 1 error:

docs/docsite/rst/community/documentation_contributions.rst:120:0: Bullet list ends without a blank line; unexpected unindent.

click here for bot help

@ansibot ansibot added ci_verified Changes made in this PR are causing tests to fail. needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. and removed core_review In order to be merged, this PR must follow the core review workflow. labels Oct 23, 2018
@ansibot
Copy link
Contributor

ansibot commented Oct 23, 2018

The test ansible-test sanity --test docs-build [explain] failed with 3 errors:

docs/docsite/rst/community/documentation_contributions.rst:120:0: warning: Bullet list ends without a blank line; unexpected unindent.
docs/docsite/rst/community/documentation_contributions.rst:169:0: undefined-label: undefined label: testing_documentation (if the link has no caption the label must precede a section header)
docs/docsite/rst/dev_guide/developing_modules_documenting.rst:126:0: undefined-label: undefined label: testing_documentation (if the link has no caption the label must precede a section header)

The test ansible-test sanity --test rstcheck [explain] failed with 1 error:

docs/docsite/rst/community/documentation_contributions.rst:120:0: Bullet list ends without a blank line; unexpected unindent.

click here for bot help

@ansibot ansibot removed the ci_verified Changes made in this PR are causing tests to fail. label Oct 23, 2018
@ansibot ansibot added core_review In order to be merged, this PR must follow the core review workflow. and removed needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. labels Oct 23, 2018

make htmlsingle rst=community/documentation_contributions.rst

This process compiles all the links but provides minimal log output. If you're writing a new page or want more detailed log output, refer to the instructions on :ref:``
Copy link
Contributor

Choose a reason for hiding this comment

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

missing what :ref: is pointing to here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Zeut alors, I have missed one!

@samccann samccann merged commit 4219d25 into ansible:devel Oct 24, 2018
Review, fix, and maintain the documentation
===========================================

Typos are everywhere, even in the Ansible documentation. We work hard to keep the documentation up-to-date, but you may also find out-dated examples. We offer easy ways to :ref:`report and/or fix documentation errors <community_documentation_contributions>`.
Copy link
Contributor

Choose a reason for hiding this comment

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

"out-dated" should be "outdated" instead.

Suggested change
Typos are everywhere, even in the Ansible documentation. We work hard to keep the documentation up-to-date, but you may also find out-dated examples. We offer easy ways to :ref:`report and/or fix documentation errors <community_documentation_contributions>`.
Typos are everywhere, even in the Ansible documentation. We work hard to keep the documentation up-to-date, but you may also find outdated examples. We offer easy ways to :ref:`report and/or fix documentation errors <community_documentation_contributions>`.

gundalow pushed a commit to gundalow/ansible that referenced this pull request Oct 26, 2018
* adds page on community contributions to docs

(cherry picked from commit 4219d25)
acozine pushed a commit that referenced this pull request Oct 26, 2018
* adds page on community contributions to docs

(cherry picked from commit 4219d25)
@acozine acozine moved this from To do to Done in OLD Ansible Documentation Nov 7, 2018
@acozine acozine deleted the contribute-to-docs branch November 29, 2018 20:39
Tomorrow9 pushed a commit to Tomorrow9/ansible that referenced this pull request Dec 4, 2018
* adds page on community contributions to docs
@ansible ansible locked and limited conversation to collaborators Jul 22, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
affects_2.8 This issue/PR affects Ansible v2.8 core_review In order to be merged, this PR must follow the core review workflow. docs This issue/PR relates to or includes documentation. support:core This issue/PR relates to code supported by the Ansible Engineering Team.
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

8 participants