Skip to content

fix(interpreter): complete runtime operations - #30

Merged
Prestgg1 merged 2 commits into
AzLangProgramingLanguage:mainfrom
martian56:fix/interpreter-operations
Jul 24, 2026
Merged

fix(interpreter): complete runtime operations#30
Prestgg1 merged 2 commits into
AzLangProgramingLanguage:mainfrom
martian56:fix/interpreter-operations

Conversation

@martian56

@martian56 martian56 commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Summary

  • implement boolean Not, And, and Or runtime behavior
  • support consistent equality and ordering for runtime values, including mixed integer/float comparisons
  • reject invalid logical and comparison operands instead of silently returning false
  • preserve numeric promotion when integer values are used in float operations

Testing

  • 38 interpreter tests passed in an isolated worktree
  • direct package testing on upstream main remains blocked by unrelated parser and validator test-module compilation issues

Closes #18

Copilot AI review requested due to automatic review settings July 22, 2026 06:34
Copilot AI reviewed Jul 22, 2026

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 was unable to review this pull request because the user who requested the review has reached their quota limit.

Prestgg1
Prestgg1 approved these changes Jul 23, 2026
View reviewed changes

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

Welcome. You are new and first contributtor.

Comment thread interpreter/src/runner/tests/binary_op.rs
Comment thread interpreter/src/runner/tests/binary_op.rs
Comment thread interpreter/src/runner/runner.rs
Copilot AI review requested due to automatic review settings July 23, 2026 07:04
Copilot AI reviewed Jul 23, 2026

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 was unable to review this pull request because the user who requested the review has reached their quota limit.

@martian56 martian56 mentioned this pull request Jul 23, 2026
Merged
@martian56

martian56 commented Jul 23, 2026

Copy link
Copy Markdown
Contributor Author

@Prestgg1 Looks good?

Prestgg1
Prestgg1 approved these changes Jul 24, 2026
View reviewed changes
Comment thread compiler/src/tests/compiler_test.rs
@Prestgg1
Prestgg1 merged commit fd51744 into AzLangProgramingLanguage:main Jul 24, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments
@Prestgg1 Prestgg1 Prestgg1 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

interpreter: implement missing binary operations (comparison, logical, unary)

3 participants

@martian56 @Prestgg1