Adding CircleCI tests for other versions of Python

[angela97lin]

No description provided.

[angela97lin]

Addresses #92

[codecov]

Codecov Report

Merging #104 into master will decrease coverage by 0.48%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #104      +/-   ##
==========================================
- Coverage   93.02%   92.54%   -0.49%     
==========================================
  Files          47       47              
  Lines        1248     1248              
==========================================
- Hits         1161     1155       -6     
- Misses         87       93       +6

Impacted Files	Coverage Δ
evalml/pipelines/classification/random_forest.py	79.31% <0%> (-20.69%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c899a91...f190b44. Read the comment docs.

[angela97lin]

(Unsure if ci/circleci: python-35 test will run since I removed it?)

[jeremyliweishih]

You can check here: https://circleci.com/gh/FeatureLabs/evalml/tree/python_circleci_tests

[angela97lin]

Right, it doesn't show up / is not queued because I've removed the test in my edits, but am confused since the merge check is still expecting it to run...

[rwedge]

The branch settings for the repo probably has that test listed as a required check

[kmax12]

Looks good to me unless @rwedge sees anything wrong

[rwedge]

This is going to upload a codecov report for each python version which may not be necessary

[rwedge]

If we want the unit tests running to not be dependent on the lint tests passing we could move adopt a larger workflow graph like featuretools uses

[rwedge]

For some reason it looks like the codecov upload isn't getting executed

[angela97lin]

Hmm... just taking a guess but since require_ci_to_pass is set to true, could it be waiting on ci/circleci: python-35 to finish running before codecov will run?

[jeremyliweishih]

maybe you can test it by setting require_ci_to_pass = false

[rwedge]

When I look at the py37 unit test container on circle ci it doesn't appear to run the codecov step at the end

What if we add an empty line at the end of the config file

[rwedge]

Looks like the codecov report is being uploaded now. Maybe we should merge it in and update the required checks?