Skip to content

mypy_test.py: Allow non-types dependencies #9408

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.

Sign up for GitHub

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
AlexWaygood merged 19 commits into from
Jan 7, 2023
Merged

mypy_test.py: Allow non-types dependencies #9408

Changes from 1 commit
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
56e755b
Refactor to use subprocess; other small improvements
AlexWaygood Dec 25, 2022
ee036e7
Allow non-types dependencies
AlexWaygood Dec 25, 2022
ca484c4
Share venvs between packages with common requirements
AlexWaygood Dec 26, 2022
db260f2
Here's what happens if we add loads of non-types requirements
AlexWaygood Dec 26, 2022
28973be
Revert "Here's what happens if we add loads of non-types requirements"
AlexWaygood Dec 26, 2022
1c80dc1
Some small improvements
AlexWaygood Dec 26, 2022
e935526
Revert `packaging` bump
AlexWaygood Dec 26, 2022
dd7a229
Move requirement-normalization logic to `utils.py`
AlexWaygood Dec 27, 2022
c5c93c9
A better if-condition in `make_venv`
AlexWaygood Dec 27, 2022
fec7b0b
Improve grammar in logged message, why not
AlexWaygood Dec 27, 2022
fc085e1
Remove redundant error code
AlexWaygood Dec 31, 2022
a590265
Add issue number to comment, for easy greppability
AlexWaygood Jan 1, 2023
bd92207
Merge branch 'main' into mypy-test-non-types
AlexWaygood Jan 3, 2023
53c3b22
Run `pip freeze` if `--verbose` is specified, there are non-types dep…
AlexWaygood Jan 3, 2023
d5f05b8
Fix
AlexWaygood Jan 3, 2023
abbd4eb
Simplify
AlexWaygood Jan 4, 2023
a363399
Merge branch 'main' into mypy-test-non-types
AlexWaygood Jan 4, 2023
82c085c
Exit early from function if no stubs have external requirements
AlexWaygood Jan 5, 2023
49b062c
Fix crash when a stubs package specifies a dependency on a non-types …
AlexWaygood Jan 5, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
A better if-condition in make_venv
  • Loading branch information
@AlexWaygood
AlexWaygood committed Dec 27, 2022
commit c5c93c920b613aabec4435916f721ae35339c0e1
2 changes: 1 addition & 1 deletion tests/utils.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -145,7 +145,7 @@ def make_venv(venv_dir: Path) -> VenvInfo:
try:
venv.create(venv_dir, with_pip=True, clear=True)
except subprocess.CalledProcessError as e:
if "ensurepip" in e.cmd and "KeyboardInterrupt" not in e.stdout:
if "ensurepip" in e.cmd and b"KeyboardInterrupt" not in e.stdout.splitlines():

@AlexWaygood AlexWaygood Dec 27, 2022

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

The "if you're on Linux, you might need to install the python3-venv package" message was showing up every time I interrupted the script with a KeyboardInterrupt during the "concurrently create many venvs" step, and it was kinda annoying.

print_error(
"stubtest requires a Python installation with ensurepip. "
"If on Linux, you may need to install the python3-venv package."
Expand Down