Skip to content

Cleanup CircleCI config #166

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

Merged

Conversation

rohitpaulk
Copy link
Contributor

@rohitpaulk rohitpaulk commented Apr 17, 2020

Includes changes from #165.

  • Moved all test dependencies to setup.py, fixes Strange Compiling error w/ make #148.
  • Ensure we're testing on the latest minor version for each major version.
  • Reduced duplication in CircleCI config by re-using config blocks

@codecov-io
Copy link

codecov-io commented Apr 17, 2020

Codecov Report

Merging #166 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #166   +/-   ##
=======================================
  Coverage   92.72%   92.72%           
=======================================
  Files           6        6           
  Lines         371      371           
  Branches       38       38           
=======================================
  Hits          344      344           
  Misses         21       21           
  Partials        6        6           

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 641862e...f296f8f. Read the comment docs.

Also replaced tests_require with extras_require. The test command (which
tests_required was used for) was deprecated in setuptools 41.5.0.
@rohitpaulk rohitpaulk force-pushed the rohitpaulk/cleanup-circleci-config branch 2 times, most recently from 8875eda to 3696afa Compare April 17, 2020 13:19
@rohitpaulk rohitpaulk force-pushed the rohitpaulk/cleanup-circleci-config branch from 3696afa to 2b3c831 Compare April 17, 2020 13:29
@rohitpaulk rohitpaulk force-pushed the rohitpaulk/cleanup-circleci-config branch 2 times, most recently from 72ab01b to ace671e Compare April 17, 2020 13:48
Includes special handling for the circleci/python:3.4 image, which
doesn't setup PATH properly. This isn't worth fixing right now, as it is
likely that we'll deprecate Python 3.4 support very soon.
@rohitpaulk rohitpaulk force-pushed the rohitpaulk/cleanup-circleci-config branch from ace671e to 84ea6ad Compare April 17, 2020 13:49
@rohitpaulk rohitpaulk marked this pull request as ready for review April 17, 2020 13:52
@lubird lubird merged commit 7681b51 into segmentio:master Apr 17, 2020
@rohitpaulk rohitpaulk deleted the rohitpaulk/cleanup-circleci-config branch April 18, 2020 07:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Strange Compiling error w/ make
3 participants