Skip to content

add except* support to 102, 103, 104, 120, 910, 911 & 912#358

Merged
jakkdl merged 3 commits intopython-trio:mainfrom
jakkdl:do_claude_things
Mar 6, 2025
Merged

add except* support to 102, 103, 104, 120, 910, 911 & 912#358
jakkdl merged 3 commits intopython-trio:mainfrom
jakkdl:do_claude_things

Conversation

@jakkdl
Copy link
Copy Markdown
Member

@jakkdl jakkdl commented Mar 5, 2025

fixes one half of #173 (the other being match/case)

First time trying out Claude Code, I more or less had to rewrite everything it generated in the end - but they gave it a good shot!

Copy link
Copy Markdown
Member

@Zac-HD Zac-HD left a comment

Choose a reason for hiding this comment

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

looking forward to this upgrade!

Comment thread tests/eval_files/async91x_py311.py Outdated
raise


# Multiple except* handlers - should all guarantee checkpoint
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

checkpoint-or-raise?

@jakkdl jakkdl merged commit ac7a9c1 into python-trio:main Mar 6, 2025
11 checks passed
@jakkdl jakkdl deleted the do_claude_things branch March 6, 2025 12:03
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.

2 participants