Skip to content

Silence sphinx warnings#500

Merged
berland merged 1 commit intomasterfrom
fix_sphinx_warnings
Sep 28, 2022
Merged

Silence sphinx warnings#500
berland merged 1 commit intomasterfrom
fix_sphinx_warnings

Conversation

@berland
Copy link
Copy Markdown
Collaborator

@berland berland commented Sep 28, 2022

Text strings that are formatted both for the command line interface and for sphinx are a little bit tricky. Tried to make it ok for both, and got rid of warnings:

None:17: ERROR: Unexpected indentation.
/home/runner/work/fmu-docs/fmu-docs/subscript/docs/scripts/runrms.rst:18: WARNING: Block quote ends without a blank line; unexpected unindent. None:25: ERROR: Unexpected indentation.
None:29: ERROR: Unexpected indentation.
/home/runner/work/fmu-docs/fmu-docs/subscript/docs/scripts/runrms.rst:30: WARNING: Block quote ends without a blank line; unexpected unindent. /home/runner/work/fmu-docs/fmu-docs/subscript/docs/scripts/runrms.rst:36:
WARNING: Definition list ends without a blank line; unexpected unindent.

Text strings that are formatted both for the command line interface
and for sphinx are a little bit tricky. Tried to make it ok for both,
and got rid of warnings:

None:17: ERROR: Unexpected indentation.
/home/runner/work/fmu-docs/fmu-docs/subscript/docs/scripts/runrms.rst:18:
WARNING: Block quote ends without a blank line; unexpected unindent.
None:25: ERROR: Unexpected indentation.
None:29: ERROR: Unexpected indentation.
/home/runner/work/fmu-docs/fmu-docs/subscript/docs/scripts/runrms.rst:30:
WARNING: Block quote ends without a blank line; unexpected unindent.
/home/runner/work/fmu-docs/fmu-docs/subscript/docs/scripts/runrms.rst:36:
WARNING: Definition list ends without a blank line; unexpected unindent.
@berland berland requested a review from jcrivenaes September 28, 2022 08:23
Copy link
Copy Markdown
Contributor

@jcrivenaes jcrivenaes left a comment

Choose a reason for hiding this comment

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

LGTM

@berland berland merged commit 51f5860 into master Sep 28, 2022
@berland berland deleted the fix_sphinx_warnings branch February 11, 2025 08:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants