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

WIP debugging travis failure #141

Closed
wants to merge 3 commits into from
Closed

WIP debugging travis failure #141

wants to merge 3 commits into from

Conversation

sjsrey
Copy link
Member

@sjsrey sjsrey commented Mar 10, 2019

No description provided.

@sjsrey sjsrey requested a review from jGaboardi March 10, 2019 18:47
@sjsrey sjsrey changed the title debugging travis failure WIP debugging travis failure Mar 10, 2019
Copy link
Member

@jGaboardi jGaboardi left a comment

Choose a reason for hiding this comment

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

@sjsrey I am just pushing nearly the same solution, but you beat me to the punch. In my branch I created a requirements_plus_conda.txt and requirements_plus_pip.txt. Do you think your method of directly doing pip install geomet in the .travis.yml or my method is the more streamlined and adaptable model?

@jGaboardi jGaboardi added rough edge Something that's not a bug, but that gets in the way of "it just works." WIP Work in progress, do not merge. Discussion only. labels Mar 10, 2019
@jGaboardi jGaboardi self-assigned this Mar 10, 2019
@sjsrey
Copy link
Member Author

sjsrey commented Mar 10, 2019

@jGaboardi not sure what is the best way. For some reason my tests are not getting triggered on my fork so i'm pr'ing into upstream to hopefully trigger a test.

@jGaboardi
Copy link
Member

@sjsrey I think yours weren't being triggered because mine were running.

@jGaboardi
Copy link
Member

or maybe not...

@jGaboardi
Copy link
Member

being addressed in #140

.travis.yml Outdated
@@ -24,7 +24,8 @@ before_install:
install:
- conda install --yes pip
- if [ "$PYSAL_PLUS" == true ]; then
echo 'plus testing'; pip install -r requirements_plus.txt;
echo 'plus testing'; conda install --yes --file requirements_plus.txt;
pip install geomat;
Copy link
Member

Choose a reason for hiding this comment

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

I think the method for having and additional requirements_plus_pip.txt is the way to go because we can list it within setup.py, like here.

@jGaboardi
Copy link
Member

@sjsrey can this be closed now?

@sjsrey sjsrey closed this May 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rough edge Something that's not a bug, but that gets in the way of "it just works." WIP Work in progress, do not merge. Discussion only.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants