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

CLIMATE-862: More updates to conda recipe files #401

Merged
merged 1 commit into from Sep 1, 2016

Conversation

agoodm
Copy link
Member

@agoodm agoodm commented Aug 29, 2016

@lewismc @Omkar20895 Please let me know if coveralls is actually necessary for the podaacpy code to run. Since it is listed as a dependency in setup.py, I also had to build a conda recipe for it internally in order to get podaacpy to build which ended up being quite a pain to deal with.

Anyway to test this, navigate to the conda_recipes directory, then run

conda update conda-build
conda build ocw
conda clean -pt
conda create -n test python
source activate test
conda install --use-local ocw

@OCWJenkins
Copy link

Can one of the admins verify this patch?

@coveralls
Copy link

Coverage Status

Coverage remained the same at 80.617% when pulling 12eb7ed on agoodm:CLIMATE-862 into ce05d38 on apache:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 80.617% when pulling 5b81f8b on agoodm:CLIMATE-862 into ce05d38 on apache:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 80.617% when pulling d47fe2e on agoodm:CLIMATE-862 into ce05d38 on apache:master.

@Omkar20895
Copy link
Contributor

Omkar20895 commented Aug 30, 2016

@agoodm No coveralls is not required for the podaacpy code to run, it just checks the total coverage the tests have on the actual code, so we are basically installing it is an add on for checking the code coverage.

@asfgit asfgit merged commit d47fe2e into apache:master Sep 1, 2016
@agoodm agoodm deleted the CLIMATE-862 branch September 1, 2016 16:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
5 participants