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 formatting of AV::TestCase.register_parser doc [ci-skip] #49924

Conversation

jonathanhefner
Copy link
Member

This fixes a few issues with the formatting of the API documentation for ActionView::TestCase::Behavior::ClassMethods#register_parser:

  • Use h4 headings instead of h3 headings. SDoc renders method signatures as h3 headings, so subheadings such as "Examples" should use h4.

  • Replace "Arguments" heading with "Parameters". "Parameters" are elements of function's signature, whereas "arguments" are elements of a function call. The API documentation for other methods follows this convention.

  • Format parameters as term list.

  • Fix indentation of pre-registered parser lists to render them as lists instead of as code.

  • Miscellaneous rewording, reorganization, and additional monospace formatting.

Before After
before after

This fixes a few issues with the formatting of the API documentation for
`ActionView::TestCase::Behavior::ClassMethods#register_parser`:

* Use `h4` headings instead of `h3` headings.  SDoc renders method
  signatures as `h3` headings, so subheadings such as "Examples" should
  use `h4`.

* Replace "Arguments" heading with "Parameters".  "Parameters" are
  elements of function's signature, whereas "arguments" are elements of
  a function call.  The API documentation for other methods follows this
  convention.

* Format parameters as term list.

* Fix indentation of pre-registered parser lists to render them as lists
  instead of as code.

* Miscellaneous rewording, reorganization, and additional monospace
  formatting.
@rails-bot rails-bot bot added the actionview label Nov 5, 2023
@jonathanhefner jonathanhefner merged commit 8a304dc into rails:main Nov 5, 2023
3 checks passed
jonathanhefner added a commit that referenced this pull request Nov 5, 2023
…rser-doc-formatting

Fix formatting of `AV::TestCase.register_parser` doc [ci-skip]

(cherry picked from commit 8a304dc)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant