Skip to content

Fix dynamic version specification. - #477

Merged
jarrodmillman merged 1 commit into
numpy:mainfrom
stefmolin:fix-version-bug
Jul 3, 2023
Merged

Fix dynamic version specification.#477
jarrodmillman merged 1 commit into
numpy:mainfrom
stefmolin:fix-version-bug

Conversation

@stefmolin

@stefmolin stefmolin commented Jul 3, 2023

Copy link
Copy Markdown
Contributor

I noticed a small bug coming from #474. In order to use the dynamic version, you have to specify where it is coming from otherwise it installs as numpydoc-0.0.0. I added the required logic to the tools/pyproject.toml.in file and ran the script to update the file.

Before change:

Successfully installed numpydoc-0.0.0

After change:

Successfully installed numpydoc-1.6.0rc1.dev0

@jarrodmillman jarrodmillman added this to the 1.6.0 milestone Jul 3, 2023
@stefanv

stefanv commented Jul 3, 2023

Copy link
Copy Markdown
Contributor

If we're compiling the toml anyway, why not inject a static version parsed from the code?

@jarrodmillman

Copy link
Copy Markdown
Member

We can make a follow-up PR if we decide to inject a static version parsed from the code.

@jarrodmillman
jarrodmillman merged commit e5cee43 into numpy:main Jul 3, 2023
@larsoner

larsoner commented Jul 3, 2023

Copy link
Copy Markdown
Collaborator

In several projects I've started using setuptools_scm with good

@larsoner

larsoner commented Jul 3, 2023

Copy link
Copy Markdown
Collaborator

... results. But you need the git history so maybe not a good choice here

@stefmolin
stefmolin deleted the fix-version-bug branch July 13, 2023 22:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants