Skip to content

Conversation

@cortadocodes
Copy link
Collaborator

@cortadocodes cortadocodes commented Jan 5, 2021

Contents

Fixes

  • Fix links to documentation in README
  • Remove generic content from README

Other fixes outside of the code

The main fixes to get the docs working were carried out in the admin interface of ReadTheDocs:

  • Untick "Install project" in the advanced settings
  • Set the default branch to main rather than master (since we've renamed our base branch)

I also stopped the building of epub and pdf docs as we're not using them.

Issues

@cortadocodes cortadocodes changed the base branch from main to release/0.1.8 January 5, 2021 17:54
@codecov-io
Copy link

codecov-io commented Jan 5, 2021

Codecov Report

Merging #71 (c486b12) into release/0.1.8 (e63b88f) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@              Coverage Diff               @@
##           release/0.1.8      #71   +/-   ##
==============================================
  Coverage          96.30%   96.30%           
==============================================
  Files                 40       40           
  Lines                839      839           
==============================================
  Hits                 808      808           
  Misses                31       31           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e63b88f...c486b12. Read the comment docs.

@cortadocodes cortadocodes requested a review from thclark January 5, 2021 19:20
@cortadocodes cortadocodes self-assigned this Jan 5, 2021
@cortadocodes cortadocodes marked this pull request as ready for review January 5, 2021 19:20
@cortadocodes cortadocodes merged commit b273b5b into release/0.1.8 Jan 6, 2021
@cortadocodes cortadocodes deleted the doc/fix-sphinx-documentation branch January 6, 2021 10:37
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.

Update SDK readme and ensure that documentation gets served

4 participants