Skip to content
This repository has been archived by the owner on Sep 8, 2024. It is now read-only.

Point sphinxdocs to new requirements.txt location #2731

Merged
merged 1 commit into from
Oct 26, 2020

Conversation

forslund
Copy link
Collaborator

Description

Sphinx docs currently fails to build the api documentation due to requirements.txt being moved. This fixes the location making the docs buildable again

How to test

Generate API docs

cd docs
make html

Check that _build/html contains updated docs.

Contributor license agreement signed?

CLA [ Yes ]

@devops-mycroft devops-mycroft added the CLA: Yes Contributor License Agreement exists (see https://github.com/MycroftAI/contributors) label Oct 23, 2020
@forslund forslund added the Type: Bug - quick Bug fixes that are quick to review and the implications of the change are clear and contained. label Oct 23, 2020
@devops-mycroft
Copy link

Voight Kampff Integration Test Succeeded (Results)

Copy link
Contributor

@krisgesling krisgesling left a comment

Choose a reason for hiding this comment

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

Good spotting, working with the fix 👍

Also like extracting it out to a variable for clarity

@krisgesling krisgesling merged commit f809ab2 into MycroftAI:dev Oct 26, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
CLA: Yes Contributor License Agreement exists (see https://github.com/MycroftAI/contributors) Type: Bug - quick Bug fixes that are quick to review and the implications of the change are clear and contained.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants