Skip to content

Conversation

@zharinov
Copy link
Member

Summary

  • Add Ord derive to Nav enum in plotnik-bytecode
  • Remove redundant NavKind wrapper from verify.rs
  • Use Nav directly in SemanticOp::Nav variant

Why

NavKind was a copy of Nav minus Epsilon, existing only because Nav lacked Ord. Adding Ord to Nav allows direct use, eliminating ~35 lines of boilerplate.

@zharinov zharinov enabled auto-merge (squash) January 15, 2026 16:32
@zharinov zharinov changed the title refactor: Remove NavKind wrapper, use Nav directly refactor: Remove NavKind, use Nav directly Jan 15, 2026
@zharinov zharinov merged commit b424a15 into master Jan 15, 2026
3 checks passed
@zharinov zharinov deleted the refactor/remove-navkind branch January 15, 2026 16:35
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