Skip to content

Update rtd 01 - #431

Merged
steven-esser merged 13 commits into
developfrom
update-rtd-01
Mar 23, 2020
Merged

steven-esser merged 13 commits into
developfrom
update-rtd-01

Conversation

@johnmhoran

Copy link
Copy Markdown
Member

No description provided.

Signed-off-by: John M. Horan <johnmhoran@gmail.com>
Signed-off-by: John M. Horan <johnmhoran@gmail.com>
Signed-off-by: John M. Horan <johnmhoran@gmail.com>
Signed-off-by: John M. Horan <johnmhoran@gmail.com>
Signed-off-by: John M. Horan <johnmhoran@gmail.com>
Signed-off-by: John M. Horan <johnmhoran@gmail.com>
Signed-off-by: John M. Horan <johnmhoran@gmail.com>
Signed-off-by: John M. Horan <johnmhoran@gmail.com>
Signed-off-by: John M. Horan <johnmhoran@gmail.com>
@johnmhoran
johnmhoran requested a review from steven-esser March 20, 2020 22:50

@steven-esser steven-esser left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This looks good, content-wise.

I left a few comments, mostly on the css override file. The repetition is only there to point out all the various instances. Other than that, this looks good.

Comment thread .gitignore Outdated
Comment thread docs/source/_static/theme_overrides.css Outdated
Comment thread docs/source/_static/theme_overrides.css Outdated
Comment thread docs/source/_static/theme_overrides.css Outdated
Comment thread docs/source/_static/theme_overrides.css Outdated
Comment thread docs/source/_static/theme_overrides.css Outdated
Comment thread docs/source/_static/theme_overrides.css Outdated
Comment thread docs/source/_static/theme_overrides.css Outdated
Comment thread docs/source/_static/theme_overrides.css Outdated
Comment thread docs/scripts/doc8_style_check.sh Outdated
Signed-off-by: John M. Horan <johnmhoran@gmail.com>
Signed-off-by: John M. Horan <johnmhoran@gmail.com>
Signed-off-by: John M. Horan <johnmhoran@gmail.com>
*  Otherwise Travis test fails because by default, Sphinx expects the
   master doc to be contents.rst, which can't be found because there is
   no such file.

Signed-off-by: John M. Horan <johnmhoran@gmail.com>
@steven-esser

Copy link
Copy Markdown
Contributor

Something if off with Travis connection to PRs, but the build have passed so I am merging this.

@steven-esser
steven-esser merged commit 314b8d5 into develop Mar 23, 2020
@steven-esser
steven-esser deleted the update-rtd-01 branch March 23, 2020 19:51
@johnmhoran

Copy link
Copy Markdown
Member Author

@MaJuRG Yes, I've been tracking this, the last Travis test passed but GH was showing it in progress, was writing a post to you.

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