Skip to content

[3.11] gh-98822: Add missing test directories to TESTSUBDIRS (#103942) #103946

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

Merged
merged 1 commit into from
Apr 27, 2023

Conversation

erlend-aasland
Copy link
Contributor

@erlend-aasland erlend-aasland commented Apr 27, 2023

@erlend-aasland
Copy link
Contributor Author

Thanks for the review, Zach. I trust the CI to wait for the macOS check 🤖 😄

@zware
Copy link
Member

zware commented Apr 27, 2023

I'm not sure it will: apparently macOS isn't required on main :)

Edit: but it is on 3.11, so that's irrelevant here 🤦‍♂️. Automerge should in fact wait here, but probably (not sure we've tested it yet?) wouldn't have in the main PR.

@erlend-aasland erlend-aasland merged commit 838bc90 into python:3.11 Apr 27, 2023
@erlend-aasland erlend-aasland deleted the 3.11-testsubdirs branch April 27, 2023 20:35
@ned-deily
Copy link
Member

ned-deily commented Apr 27, 2023

I will explicitly test this on both main and 3.11 on macOS. I just noticed on Tuesday at the sprint just before I left that no sqlite tests seemed to be running when using an installed Python and was going to investigate that next; I suspected it would turn out to be something like this.

zware added a commit to zware/cpython that referenced this pull request Apr 28, 2023
A few differences from the main branch were missed in pythonGH-103946.
zware added a commit that referenced this pull request Apr 28, 2023
A few differences from the main branch were missed in GH-103946.
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