Skip to content

fix license values in pyproject.toml#349

Closed
newville wants to merge 2 commits intomasterfrom
pyproject_fixes
Closed

fix license values in pyproject.toml#349
newville wants to merge 2 commits intomasterfrom
pyproject_fixes

Conversation

@newville
Copy link
Member

@newville newville commented Jan 9, 2026

fixes license settings in pyproject.toml to comply with the latest PyPA recommendations/requirements.

  • Closes # (insert issue number)
  • Executed pre-commit run --all-files with no errors
  • The change is fully covered by automated unit tests
  • Documented in docs/ as appropriate
  • Added an entry to the CHANGES file

@codecov
Copy link

codecov bot commented Jan 9, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.17%. Comparing base (3045fe4) to head (12d1cc2).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #349   +/-   ##
=======================================
  Coverage   87.17%   87.17%           
=======================================
  Files          20       20           
  Lines        2019     2019           
=======================================
  Hits         1760     1760           
  Misses        259      259           
Flag Coverage Δ
macos-latest-3.10 95.64% <ø> (ø)
macos-latest-3.11 95.64% <ø> (ø)
macos-latest-3.12 95.64% <ø> (ø)
macos-latest-3.13 95.64% <ø> (ø)
macos-latest-3.14 95.64% <ø> (ø)
macos-latest-3.8 ?
macos-latest-3.9 95.64% <ø> (ø)
no-numpy 82.51% <ø> (ø)
ubuntu-latest-3.10 95.64% <ø> (ø)
ubuntu-latest-3.11 95.64% <ø> (ø)
ubuntu-latest-3.12 95.64% <ø> (ø)
ubuntu-latest-3.13 95.64% <ø> (ø)
ubuntu-latest-3.14 95.64% <ø> (ø)
ubuntu-latest-3.8 ?
ubuntu-latest-3.9 95.64% <ø> (ø)
windows-latest-3.10 95.64% <ø> (ø)
windows-latest-3.11 95.64% <ø> (ø)
windows-latest-3.12 95.64% <ø> (ø)
windows-latest-3.13 95.64% <ø> (ø)
windows-latest-3.14 95.64% <ø> (ø)
windows-latest-3.8 ?
windows-latest-3.9 95.64% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@codspeed-hq
Copy link

codspeed-hq bot commented Jan 9, 2026

Merging this PR will not alter performance

✅ 5 untouched benchmarks


Comparing pyproject_fixes (12d1cc2) with master (3045fe4)

Open in CodSpeed

@newville
Copy link
Member Author

newville commented Jan 9, 2026

@wshanks, @jagerber48 This fixes the pyproject.toml file so that PyPI and python -m build does not complain, but that requires dropping Python 3.8. FWIW, we could drop Python 3.9, but this does not do that yet.

@wshanks
Copy link
Contributor

wshanks commented Jan 9, 2026

@newville What do you think? Should we try restarting the 3.2.4 publish workflow one time first to see if it goes through and then merge this in preparation for 4.0? Or try to get this into a revised 3.2.4? I don't object to anything here but I just wonder about if it should make it into 3.2.4 if it is not needed for the publish job to run.

@newville
Copy link
Member Author

newville commented Jan 9, 2026

@wshanks @jagerber48 Yes, let's try it one more time. If it fails, I can build locally and should be able to push to PyPI.

Copy link
Contributor

@wshanks wshanks left a comment

Choose a reason for hiding this comment

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

Okay, I tried switching to a rule in the pypi-publish environment in the Environments section that restricted publishing to tags matching the pattern [0-9][0-9]*.[0-9][0-9]*.[0-9][0-9]* which some things I found on the internet said was the right pattern for this (fnmatch syntax) but I think they might have been wrong. I still got the same error about 3.2.4 not being allowed to publish. Rather than fiddle with that for now I just deleted the rule to allow publishing from any branch or tag and reran the workflow again and that time it seems to have worked. If anyone is interested, they can try to put the right syntax in to limit publishing only to the tags we expect to publish.

- Adjusts the Sphinx configuration to allow for reproducible builds using
``SOURCE_DATE_EPOCH``.
- Mark Python 3.13 and 3.14 as officially supported.
- Drop support for Python 3.8
Copy link
Contributor

Choose a reason for hiding this comment

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

This has to move out of the 3.2.4 section now.

Copy link
Member Author

Choose a reason for hiding this comment

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

@wshanks where should it go, then?

Copy link
Contributor

Choose a reason for hiding this comment

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

3.2.4 is released. We need a new unreleased section and it will eventually go into, probably, 3.2.5. Or if this is the only change we want in 3.2.5 you can make a 3.2.5 section put it in there and we can release again today or tomorrow.

Copy link
Member Author

Choose a reason for hiding this comment

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

Totally baffled. Did someone push 3.2.4 to PyPI? I did not.
The changes to pyproject.toml should be made at some point.

Closing.

Copy link
Contributor

Choose a reason for hiding this comment

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

Sorry for the confusion. When I asked this:

Should we try restarting the 3.2.4 publish workflow one time first to see if it goes through and then merge this in preparation for 4.0?

I meant "should we try triggering the 3.2.4 publish workflow resulting in 3.2.4 being successfully published?" So that was what I tried and it did result in the previously identified commit (current master) being published as 3.2.4.

Copy link
Member Author

Choose a reason for hiding this comment

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

OK, thanks.

Copy link
Contributor

@jagerber48 jagerber48 Jan 12, 2026

Choose a reason for hiding this comment

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

The changes to pyproject.toml should be made at some point.

Can we just re-open this PR to keep the issue tracked so we don't forget about it? Totally independent of 4.0 timeline, just don't want to lose track of it since you've already done work for it.

Copy link
Member Author

Choose a reason for hiding this comment

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

Unfortunately, this didn't get put into 3.2.4. But I will leave that up to you and @wshanks.... still a bit miffed that I started working on this untagged 3.2.4, and then3.2.4 got released despite working to fix this and waiting for comments.

Copy link
Contributor

Choose a reason for hiding this comment

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

Hmm, I thought my message here was pretty clear that I was asking about trying to release 3.2.4 without the changes in this PR because they were not necessary for the release of 3.2.4 that we had already agreed on releasing (GitHub releases are tied to git tags).

Over the last year, my use of Uncertainties has been very minimal, so my efforts here are just recreational because I find it fun to maintain code that other people find useful. The amount of activity on this repo that evokes a negative emotion in me too high for a recreational project. There are enough other maintainers and you are more than capable of keeping this project in good shape. I am going to withdraw from maintainership at this time.

@newville newville closed this Jan 10, 2026
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.

3 participants