Skip to content
This repository has been archived by the owner on Mar 25, 2021. It is now read-only.

Kilo.travis fix for docs.161 #163

Closed

Conversation

ssorenso
Copy link

Fixing .travis.yml issues regarding docs

Issues:
WIP #161

Problem:

  • Travis does not install the requirements.docs.txt prior to executing
    docs

Analysis:

  • This change will have the travis instance install with -e ./
    • This will allow the importation of f5_heat

Tests:

  • This is for the test suite.

Issues:
WIP F5Networks#161

Problem:
* Travis does not install the requirements.docs.txt prior to executing
    docs

Analysis:
* This change will have the travis instance install with `-e ./`
  * This will allow the importation of f5_heat

Tests:
* This is for the test suite.
@ssorenso
Copy link
Author

@pjbreaux @jputrino Please review.

@jputrino, this should fix your docs problem.

@pjbreaux
Copy link
Contributor

@ssorenso: look at the above message about kilo branch being out-of-date

@ssorenso ssorenso changed the base branch from master to kilo October 12, 2017 15:55
@jputrino
Copy link
Contributor

@ssorenso @pjbreaux I actually have a fix for this already in PR #162. The pip install has to happen in the docs test script because it runs in a container and doesn't have access to the requirements that we already installed in the travis build environment. I had it set up this way before, I just accidentally removed it when I was updating the script.

@ssorenso
Copy link
Author

Will remove the PR then. Thanks to you both for the information.

@ssorenso ssorenso closed this Oct 12, 2017
@ssorenso ssorenso deleted the kilo.travis_fix_for_docs.161 branch October 12, 2017 16:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants