Skip to content

Don't use setuptools_scm: revert to manual version management#586

Merged
fredrik-johansson merged 1 commit into
mpmath:masterfrom
skirpichev:no-sscm
Mar 26, 2021
Merged

Don't use setuptools_scm: revert to manual version management#586
fredrik-johansson merged 1 commit into
mpmath:masterfrom
skirpichev:no-sscm

Conversation

@skirpichev

Copy link
Copy Markdown
Collaborator

The setuptools_scm usage was broken by 867d878 (likely,
this is a source of problem in #585)

Closes #580 (an alternative to #583 and #581)
Solves #585 as well.

The setuptools_scm usage was broken by 867d878 (likely,
this is a source of problem in #585)

Closes #580 (an alternative to #583 and #581)
Solves #585 as well.
@mikedh

mikedh commented Mar 26, 2021

Copy link
Copy Markdown

This looks like a great solution! 👍

@fredrik-johansson fredrik-johansson merged commit bd6715d into mpmath:master Mar 26, 2021
@skirpichev skirpichev deleted the no-sscm branch March 26, 2021 22:02
@skirpichev

Copy link
Copy Markdown
Collaborator Author

I'm not so sure. setuptools_scm is a good solution for automation of trivial tasks. Now maintainer will have to keep in sync __version__ and git tag again. For no reason, actually (#443 has no real problems, but it's "solution" #446 introduce a new one: #585). py2 issue may be solved for setuptools_scm-based version management (#583).

Anyway, probably #583, #581 and #585 could be closed now.

@mikedh

mikedh commented Mar 26, 2021

Copy link
Copy Markdown

Agreed, I think this fixes this issue. FWIW I tag-on-release using a Github action which works pretty well.

@skirpichev

Copy link
Copy Markdown
Collaborator Author

Should work too, why not. Not sure if it's possible to sign such a tag, however.

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.

setuptools_scm dropped Python 2 breaking install

3 participants