Conversation
Hello @dkliban! Thanks for updating the PR. Cheers ! There are no PEP8 issues in this Pull Request. 🍻 Comment last updated on July 13, 2018 at 15:09 Hours UTC |
9af166b
to
5569e36
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I haven't tested this (and I'm not sure that it can be tested easily to begin with) but this all looks reasonable to me.
.travis.yml
Outdated
@@ -1,19 +1,27 @@ | |||
language: python | |||
python: | |||
- "2.7" | |||
- '3.6' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor grip but typically arrays in yaml are indented.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In the files below (e.g. .travis/2-master.yaml
) you're indenting array entries.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Small comment and a question.
@@ -0,0 +1,13 @@ | |||
Documentation Index |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I feel like these files aren't necessarily specific to Travis? Thoughts on maybe moving these to the top level docs directory?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
They are specific to the way we build docs for docs.pulpproject.org. You can build the docs in this repo by themselves and they will not include plugin information.
5569e36
to
15a6eed
Compare
The 2-master branch will be setup to build using cron on Travis. When this build is trigerred, the docs for branches defined in .travis/2-master.yaml will be built and published. re: pulp#3722 https://pulp.plan.io/issues/3722
15a6eed
to
2259509
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
The 2-master branch will be setup to build using cron on Travis. When this build is trigerred, the
docs for branches defined in .travis/2-master.yaml will be built and published.
re: #3722
https://pulp.plan.io/issues/3722