Update recipe to 3.2.0#9
Conversation
|
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( |
|
@conda-forge-admin, please rerender |
|
@jakirkham I used all the suggestions mentioned here but the SSL issue isn't fixed. Is there something I am doing wrong? |
| - forge_test.sh | ||
| commands: | ||
| - time bash ./forge_test.sh # [not win] | ||
| - nosetests --exe -v gensim |
There was a problem hiding this comment.
This would need to move into the shell script to work.
|
@menshikh-iv The PR supersedes #8. Would you like to take a look and see if any changes need to be made? |
There was a problem hiding this comment.
In Appveyor, I see that tests aren't executed, why @souravsingh?
| - scikit-learn | ||
| - pyemd | ||
| - mock | ||
| - boto3 |
There was a problem hiding this comment.
boto3 isn't gensim dependency
There was a problem hiding this comment.
For some reason, the import tests throw an error due to boto3 dependency for smart_open. That's why I added it.
| # Add path, activate `conda` and update conda. | ||
| - cmd: call %CONDA_INSTALL_LOCN%\Scripts\activate.bat | ||
| - cmd: conda update --yes --quiet conda | ||
| - cmd: conda.exe update --yes --quiet conda |
There was a problem hiding this comment.
only question - what's a reason for it?
There was a problem hiding this comment.
This commit is made by the conda-forge admin. It is just for updating the testing for Windows platform and to update to latest version of Conda for testing.
| @@ -0,0 +1,10 @@ | |||
| #! /bin/bash | |||
There was a problem hiding this comment.
Why you "hide" testing in this file?
There was a problem hiding this comment.
The script contains environment variables for SSL_CERT_FILE and REQUESTS_CA_BUNDLE, without unsetting them beforehand, the downloader tests fail. So I used the solution used by people here for running the test suite for Linux.
|
Regarding Appveyor tests, The CI just runs the import tests. The latest commit should fix it. |
|
Thanks @souravsingh 👍 |
5 participants
No description provided.