Skip to content

impl code action idea: if else chain to match case #3017 - #3409

Closed
asukaminato0721 wants to merge 2 commits into
meta:mainfrom
asukaminato0721:3017
Closed

asukaminato0721 wants to merge 2 commits into
meta:mainfrom
asukaminato0721:3017

Conversation

@asukaminato0721

Copy link
Copy Markdown
Collaborator

Summary

Fixes #3017

Test Plan

@meta-cla meta-cla Bot added the cla signed label May 15, 2026
@github-actions github-actions Bot added size/xl and removed size/xl labels May 15, 2026
@github-actions

This comment has been minimized.

@asukaminato0721
asukaminato0721 marked this pull request as ready for review May 15, 2026 18:57
Copilot AI review requested due to automatic review settings May 15, 2026 18:57

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@meta-codesync

meta-codesync Bot commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

This pull request has been imported. If you are a Meta employee, you can view this in D114357319. (Because this pull request was imported automatically, there will not be any future comments.)

@github-actions

This comment has been minimized.

@github-actions

Copy link
Copy Markdown

According to mypy_primer, this change doesn't affect type check results on a corpus of open source code. ✅

@stroxler stroxler assigned stroxler and unassigned connernilsen Oct 30, 2026
@github-actions

Copy link
Copy Markdown

Closing this PR because linked issue #3017 was closed.

@github-actions github-actions Bot closed this Oct 30, 2026
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.

code action idea: if else chain to match case

4 participants