Skip to content

fix(if-else): stop legacy fallback on case miss#44

Merged
WH-2099 merged 1 commit intomainfrom
fix/if-else-stop-legacy-fallback
Apr 16, 2026
Merged

fix(if-else): stop legacy fallback on case miss#44
WH-2099 merged 1 commit intomainfrom
fix/if-else-stop-legacy-fallback

Conversation

@WH-2099
Copy link
Copy Markdown
Member

@WH-2099 WH-2099 commented Apr 16, 2026

Important

  1. Make sure you have read our contribution guidelines
  2. Search existing issues and pull requests to confirm this change is not a duplicate
  3. Open or identify the issue this pull request resolves or advances
  4. Use a Conventional Commits title for this pull request, and mark breaking changes with !
  5. Remember that the pull request title will become the squash merge commit message
  6. If CLA Assistant prompts you, sign CLA.md in the pull request conversation

Related Issue

Closes #43

Summary

  • stop falling back to deprecated top-level conditions when cases are present but none match
  • treat cases=[] as an explicit new-shape configuration instead of a legacy signal
  • restructure the if_else tests around runtime behavior, shape priority, short-circuiting, warnings, and variable mapping

Checklist

  • This pull request links the issue it resolves or advances
  • This pull request title follows Conventional Commits, and any breaking change is marked with !
  • If CLA Assistant prompted me, I signed CLA.md in the pull request conversation

@WH-2099 WH-2099 self-assigned this Apr 16, 2026
@WH-2099 WH-2099 marked this pull request as ready for review April 16, 2026 21:26
@dosubot dosubot Bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Apr 16, 2026
@WH-2099 WH-2099 enabled auto-merge (squash) April 16, 2026 21:26
@WH-2099 WH-2099 merged commit dcd3d64 into main Apr 16, 2026
5 checks passed
@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Apr 16, 2026
@WH-2099 WH-2099 deleted the fix/if-else-stop-legacy-fallback branch April 16, 2026 21:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(if-else): stop legacy fallback on case miss

2 participants