Skip to content

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

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

jakkdl merged 3 commits into
python-trio:mainfrom
jakkdl:do_claude_things

Conversation

@jakkdl

@jakkdl jakkdl commented Mar 5, 2025

Copy link
Copy Markdown
Member

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!

@Zac-HD Zac-HD left a comment

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.

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
@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