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

ci/cd: Get rid of race condition #56

Open
JasonYao opened this issue Feb 6, 2019 · 0 comments
Open

ci/cd: Get rid of race condition #56

JasonYao opened this issue Feb 6, 2019 · 0 comments
Labels
Easiness 1 Small patch or refactor or bug fix Priority 1 Second highest priority

Comments

@JasonYao
Copy link
Owner

JasonYao commented Feb 6, 2019

Info

  • Back with the change in 8bfea51, one of the changes was adding in a matrix of supported python versions, namely 3.6 and 3.7
  • However, now that we have two environments, there is now a race condition in the deploy step, where both try to upload.
  • Now this isn't that big of an issue, since if it already finds that it exists, then it gracefully exits, but it'd still be a good thing to only do a release on one environment.
  • Recommendation is to do it on the 3.7 one, or at least a 3.7 one
@JasonYao JasonYao added Priority 1 Second highest priority Easiness 1 Small patch or refactor or bug fix labels Feb 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Easiness 1 Small patch or refactor or bug fix Priority 1 Second highest priority
Projects
None yet
Development

No branches or pull requests

1 participant