Skip to content

Conversation

@bkeryan
Copy link
Collaborator

@bkeryan bkeryan commented Apr 26, 2025

What does this Pull Request accomplish?

Single-source typing backports in a single submodule.

Why should this Pull Request be merged?

Don't repeat yourself.

Avoid accidentally depending on typing_extensions at run time.

What testing has been done?

Ran lint, mypy, and tests on Python 3.11.9

@bkeryan bkeryan requested a review from csjall as a code owner April 26, 2025 16:10
@bkeryan bkeryan requested a review from mikeprosserni April 26, 2025 16:10
@bkeryan bkeryan force-pushed the users/bkeryan/append-waveform branch from 3caa239 to 18a9000 Compare April 26, 2025 16:30
@bkeryan bkeryan force-pushed the users/bkeryan/refactor-typing branch from d176490 to 86475a1 Compare April 26, 2025 16:30
@bkeryan
Copy link
Collaborator Author

bkeryan commented Apr 28, 2025

FYI @csjall @mikeprosserni This was chained to my append waveform PR but I'm rebasing it against main so I can merge it sooner.

@bkeryan bkeryan changed the base branch from users/bkeryan/append-waveform to main April 28, 2025 23:24
@bkeryan bkeryan force-pushed the users/bkeryan/refactor-typing branch from 86475a1 to bf343e8 Compare April 28, 2025 23:25
@github-actions
Copy link
Contributor

github-actions bot commented Apr 28, 2025

Test Results

   10 files  ±0     10 suites  ±0   19s ⏱️ -1s
  268 tests ±0    263 ✅ ±0   5 💤 ±0  0 ❌ ±0 
2 680 runs  ±0  2 630 ✅ ±0  50 💤 ±0  0 ❌ ±0 

Results for commit a21ddf9. ± Comparison against base commit ad552a3.

♻️ This comment has been updated with latest results.

@bkeryan bkeryan merged commit f33ea2f into main Apr 28, 2025
13 checks passed
@bkeryan bkeryan deleted the users/bkeryan/refactor-typing branch April 28, 2025 23:38
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