Skip to content

fix: always pull versions from metadata#782

Merged
henryiii merged 1 commit intowntrblm:mainfrom
henryiii:henryiii/fix/metadataversion
Feb 24, 2024
Merged

fix: always pull versions from metadata#782
henryiii merged 1 commit intowntrblm:mainfrom
henryiii:henryiii/fix/metadataversion

Conversation

@henryiii
Copy link
Copy Markdown
Collaborator

Close #779. A random attribute can move around, but we can always ask for the metadata version.

Signed-off-by: Henry Schreiner <henryschreineriii@gmail.com>
@chrysle
Copy link
Copy Markdown
Contributor

chrysle commented Feb 24, 2024

Have you already considered adding a compat module?

@henryiii
Copy link
Copy Markdown
Collaborator Author

Of course. https://learn.scientific-python.org/development/patterns/backports/

I’ll probably do that in the future.

@henryiii henryiii merged commit 55e09cd into wntrblm:main Feb 24, 2024
@henryiii henryiii deleted the henryiii/fix/metadataversion branch February 24, 2024 15:08
@henryiii
Copy link
Copy Markdown
Collaborator Author

@chrysle though, since I expect to see if we are okay dropping 3.7 after the next release, and basically every comparison I see is based on < 3.8, it might not be worth adding a compat module. I'll probably wait to see after next release.

@chrysle
Copy link
Copy Markdown
Contributor

chrysle commented Feb 24, 2024

I'll probably wait to see after next release.

Sounds good. If you need some help, I'm available for hire ;-)

https://learn.scientific-python.org/development/patterns/backports/

Good article, by the way.

@edgarrmondragon
Copy link
Copy Markdown
Contributor

fwiw there's still ~23% of Nox downloads on 3.7, though I guess users could just use an older Nox if they're stuck on an EOL Python

a87b0a8e-22a4-4cb6-bd3e-d7e2bd423c5f.png

(Disregard the August downloads, they're incomplete)

@henryiii
Copy link
Copy Markdown
Collaborator Author

though I guess users could just use an older Nox if they're stuck on an EOL Python

Yep. Though that’s why I’d like to get this release out supporting 3.7, since we have already put the effort in. In general, it’s not always helpful to keep supporting EoL Pythons, as you can easily break something that would have worked if you just dropped the older Python. But the last release should be a good one. :)

github-actions Bot pushed a commit to msclock/sphinx-deployment that referenced this pull request Mar 4, 2024
## [0.0.20](v0.0.19...v0.0.20) (2024-03-04)

### Chores

* **deps:** bump wntrblm/nox from 2023.04.22 to 2024.03.02 ([#57](#57)) ([d177375](d177375)), closes [wntrblm/nox#762](wntrblm/nox#762) [wntrblm/nox#787](wntrblm/nox#787) [wntrblm/nox#730](wntrblm/nox#730) [wntrblm/nox#780](wntrblm/nox#780) [wntrblm/nox#770](wntrblm/nox#770) [wntrblm/nox#707](wntrblm/nox#707) [wntrblm/nox#687](wntrblm/nox#687) [wntrblm/nox#756](wntrblm/nox#756) [wntrblm/nox#652](wntrblm/nox#652) [wntrblm/nox#712](wntrblm/nox#712) [wntrblm/nox#781](wntrblm/nox#781) [wntrblm/nox#786](wntrblm/nox#786) [wntrblm/nox#684](wntrblm/nox#684) [wntrblm/nox#723](wntrblm/nox#723) [wntrblm/nox#725](wntrblm/nox#725) [wntrblm/nox#714](wntrblm/nox#714) [wntrblm/nox#715](wntrblm/nox#715) [wntrblm/nox#696](wntrblm/nox#696) [wntrblm/nox#774](wntrblm/nox#774) [wntrblm/nox#782](wntrblm/nox#782) [wntrblm/nox#722](wntrblm/nox#722) [wntrblm/nox#724](wntrblm/nox#724) [wntrblm/nox#721](wntrblm/nox#721) [wntrblm/nox#744](wntrblm/nox#744) [wntrblm/nox#738](wntrblm/nox#738) [wntrblm/nox#762](wntrblm/nox#762) [wntrblm/nox#787](wntrblm/nox#787) [wntrblm/nox#730](wntrblm/nox#730) [wntrblm/nox#780](wntrblm/nox#780) [wntrblm/nox#770](wntrblm/nox#770) [wntrblm/nox#707](wntrblm/nox#707) [wntrblm/nox#687](wntrblm/nox#687) [wntrblm/nox#756](wntrblm/nox#756) [wntrblm/nox#652](wntrblm/nox#652) [wntrblm/nox#712](wntrblm/nox#712) [wntrblm/nox#781](wntrblm/nox#781) [wntrblm/nox#786](wntrblm/nox#786) [wntrblm/nox#684](wntrblm/nox#684) [wntrblm/nox#723](wntrblm/nox#723) [wntrblm/nox#725](wntrblm/nox#725) [wntrblm/nox#714](wntrblm/nox#714) [wntrblm/nox#715](wntrblm/nox#715) [wntrblm/nox#696](wntrblm/nox#696) [wntrblm/nox#774](wntrblm/nox#774) [wntrblm/nox#782](wntrblm/nox#782) [wntrblm/nox#722](wntrblm/nox#722) [wntrblm/nox#724](wntrblm/nox#724) [wntrblm/nox#721](wntrblm/nox#721) [wntrblm/nox#744](wntrblm/nox#744) [#789](https://github.com/msclock/sphinx-deployment/issues/789) [#787](https://github.com/msclock/sphinx-deployment/issues/787) [#786](https://github.com/msclock/sphinx-deployment/issues/786) [#781](https://github.com/msclock/sphinx-deployment/issues/781) [#784](https://github.com/msclock/sphinx-deployment/issues/784) [#780](https://github.com/msclock/sphinx-deployment/issues/780) [#730](https://github.com/msclock/sphinx-deployment/issues/730) [#782](https://github.com/msclock/sphinx-deployment/issues/782) [#783](https://github.com/msclock/sphinx-deployment/issues/783) [#762](https://github.com/msclock/sphinx-deployment/issues/762)

### CI

* set proper permission for preview job ([#56](#56)) ([d65e8a1](d65e8a1))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

'virtualenv.version' has no attribute 'version' in test_virtualenv.py

4 participants