Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update to pybind11-stubgen 2.x #211

Merged
merged 12 commits into from
Oct 15, 2023
Merged

Update to pybind11-stubgen 2.x #211

merged 12 commits into from
Oct 15, 2023

Conversation

auscompgeek
Copy link
Member

@auscompgeek auscompgeek commented Aug 29, 2023

@auscompgeek
Copy link
Member Author

We probably want to use pybind11-stubgen 1.0 when it's released, given the bunch of bugfixes it'll have. Looks like that means dropping support for Python < 3.8.

@auscompgeek auscompgeek mentioned this pull request Aug 29, 2023
@auscompgeek
Copy link
Member Author

Well, I'm surprised it wasn't a lot of effort to get that to work.

@auscompgeek auscompgeek marked this pull request as ready for review August 29, 2023 14:57
@auscompgeek
Copy link
Member Author

... wtf windows

@@ -67,8 +67,6 @@ jobs:
matrix:
os: [windows-latest, macos-latest, ubuntu-20.04]
python_version:
- '3.6'
- '3.7'

Choose a reason for hiding this comment

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

pybind11-stubgen>=1.2 supports Python 3.7.
3.6 would require me to get rid of from __future__ import annotations, which would cause too much of a burden to my code.

@auscompgeek auscompgeek changed the title Update to pybind11-stubgen 1.0.dev1 Update to pybind11-stubgen 1.2 Oct 9, 2023
@auscompgeek auscompgeek changed the title Update to pybind11-stubgen 1.2 Update to pybind11-stubgen 2.x Oct 9, 2023
setup.cfg Show resolved Hide resolved
Copy link
Member

@virtuald virtuald left a comment

Choose a reason for hiding this comment

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

LGTM. Are you going to rebase the history or squash it?

@auscompgeek
Copy link
Member Author

I'll probably just squash this, unless you think any of this commit history is useful.

@virtuald virtuald merged commit 51950e6 into main Oct 15, 2023
22 checks passed
@virtuald virtuald deleted the stubgen-1.0 branch October 15, 2023 02:14
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