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

Separate release into build and release steps #858

Merged

Conversation

hoylemd
Copy link
Contributor

@hoylemd hoylemd commented Aug 2, 2018

This allows developers to build the project without publishing it to pypi.

Also add the -f tag to the cleanup bit so it doesn't bail when dist/ is missing.

Also add the -f tag to the cleanup bit so it doesn't bail when `dist/`
is missing
@codecov
Copy link

codecov bot commented Aug 2, 2018

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #858   +/-   ##
=======================================
  Coverage   66.55%   66.55%           
=======================================
  Files          14       14           
  Lines        1438     1438           
  Branches      226      226           
=======================================
  Hits          957      957           
  Misses        430      430           
  Partials       51       51

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 d53618b...289ad55. Read the comment docs.

Copy link
Member

@cgoldberg cgoldberg left a comment

Choose a reason for hiding this comment

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

LGTM

@cgoldberg cgoldberg merged commit 6701f70 into locustio:master Aug 2, 2018
spayeur207 pushed a commit to spayeur207/locust that referenced this pull request Sep 12, 2018
Also add the -f tag to the cleanup bit so it doesn't bail when `dist/`
is missing
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.

2 participants