Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Integrate travis-sphinx tool and remove doc_upload.py #57

Merged
merged 2 commits into from
Aug 17, 2015
Merged

Integrate travis-sphinx tool and remove doc_upload.py #57

merged 2 commits into from
Aug 17, 2015

Conversation

Syntaf
Copy link
Member

@Syntaf Syntaf commented Aug 17, 2015

I finished a lightweight tool that can be installed and called via pip, so I went ahead and changed this implementation which cleaned up a couple things. I also removed the encrypted token as you can actually specify this manually on travis-ci's page. This is found in the settings tab

img

Jwely added a commit that referenced this pull request Aug 17, 2015
Integrate travis-sphinx tool and remove doc_upload.py
@Jwely Jwely merged commit b2a212f into NASA-DEVELOP:master Aug 17, 2015
@Jwely
Copy link
Member

Jwely commented Aug 17, 2015

This is really neat. I'll be testing everything out and updating the documentation pages. I'll let you know if it slugs and compile some of the instructions you've put together in one place on the developers pages.

@Syntaf
Copy link
Member Author

Syntaf commented Aug 17, 2015

thanks, it's currently running on travis-ci's legacy infrastructure which means it can take a couple minutes to finish a build. legacy supports the use of sudo which is currently used in the script

It would be better to upgrade to their current infrastructure(which guarantee's a build within 10 seconds of pushing a commit) however I cannot find a workaround for the use of sudo on line 9 and 10 in .travis.yml. Maybe i'll find a fix in the future

@Jwely
Copy link
Member

Jwely commented Aug 18, 2015

Great, i checked the logs and saw the warning but figured you'd already noticed. It's a nice to-do item. I've created an issue at #58

feel free to edit for correctness.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants