PCMSolver / pcmsolver

An API for the Polarizable Continuum Model
http://pcmsolver.readthedocs.io/
GNU Lesser General Public License v3.0
32 stars 21 forks source link

Failing readthedocs #163

Closed arnfinn closed 6 years ago

arnfinn commented 6 years ago

Failed with this commit: 809e4a9a

Reason: Line 83 in doc/conf.py throws an error since prere is NoneType.

Error:

  File "conf.py", line 83, in <module>
    tweak = '+'.join([prere, sha])
TypeError: sequence item 0: expected str instance, NoneType found

Dirty fix:

-tweak = '+'.join([prere, sha])
+try:
+    tweak = '+'.join([prere, sha])
+except:
+    tweak = sha
robertodr commented 6 years ago

Yep. I am on it. I will also have to add "check that docs build" to the release management documentation.

On Fri, Apr 27, 2018, 17:15 Arnfinn Hykkerud Steindal < notifications@github.com> wrote:

Failed with this commit: 809e4a9 https://github.com/PCMSolver/pcmsolver/commit/809e4a9a6013edb35728d041a0ab796133ec07b7

Reason: Line 83 in doc/conf.py throws an error since prere is NoneType.

Error:

File "conf.py", line 83, in tweak = '+'.join([prere, sha]) TypeError: sequence item 0: expected str instance, NoneType found

Dirty fix:

-tweak = '+'.join([prere, sha]) +try:

  • tweak = '+'.join([prere, sha]) +except:
  • tweak = sha

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/PCMSolver/pcmsolver/issues/163, or mute the thread https://github.com/notifications/unsubscribe-auth/ADiXES-R0oyyNFfFnaTd7X7gztfqIjKbks5ts4p5gaJpZM4TrBl1 .

arnfinn commented 6 years ago

Do we run sphinx-build . _build with Travis to see if a PR/commit break readthedocs? If not, maybe we could add it. Maybe also run htmlproofer afterwards.

arnfinn commented 6 years ago

Fixed here: e882df9275