scikit-hep / pyhf

pure-Python HistFactory implementation with tensors and autodiff
https://pyhf.readthedocs.io/
Apache License 2.0
283 stars 83 forks source link

Test Deploy feature is broken #432

Closed kratsg closed 5 years ago

kratsg commented 5 years ago

Description

See https://github.com/FaradayRF/faradayio/issues/35 for a related issue - I think. Failing job on master here: https://travis-ci.org/diana-hep/pyhf/builds/517678508?utm_source=github_status&utm_medium=notification

Expected Behavior

Expect it to pass.

Actual Behavior

Observe it failing.

Checklist

matthewfeickert commented 5 years ago

This is behaving as expected and isn't a bug. The PR was merged in before I had finished it.

See FaradayRF/faradayio#35 for a related issue - I think.

No, this issue is a misunderstanding how PyPI works in that you can only have 1 tag associated with a repo deployed ever even if that repo is deleted. This is why the CI is failing as it is trying to deploy a tag that already exists, which isn't what I intended.

matthewfeickert commented 5 years ago

ansible-lint is able to do this properly, but that's apparently because they use setuptools-scm. I won't be able to get to this until tonight, so @kratsg if you have some PRs that you want to get done before then feel free to revert PR #429 (or just know that the last bit will error even if everything else is fine).