Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Check type at assignment #1239

Merged
merged 1 commit into from
Feb 20, 2025
Merged

Check type at assignment #1239

merged 1 commit into from
Feb 20, 2025

Conversation

dalance
Copy link
Collaborator

@dalance dalance commented Feb 11, 2025

Closes #1012
Closes #790

This PR adds the following checks:

  • 4-state value assignment to 2-state variable
  • inconsistent array dimension around assignment
  • Invalid select like wrong index order and out of range

@dalance dalance added the enhancement New feature or request label Feb 11, 2025
Copy link

codspeed-hq bot commented Feb 11, 2025

CodSpeed Performance Report

Merging #1239 will not alter performance

Comparing type_assignment (87ba73e) with master (999d6eb)

Summary

✅ 3 untouched benchmarks

@dalance dalance force-pushed the type_assignment branch 8 times, most recently from 77379fb to 5b731ab Compare February 20, 2025 03:25
@dalance dalance marked this pull request as ready for review February 20, 2025 03:36
@dalance dalance added this to the v0.13.6 milestone Feb 20, 2025
@dalance dalance force-pushed the type_assignment branch 2 times, most recently from 12988a7 to 2033402 Compare February 20, 2025 07:13
@dalance dalance merged commit cbb7554 into master Feb 20, 2025
10 checks passed
@dalance dalance deleted the type_assignment branch February 20, 2025 07:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Variable assignment without data type validation Quad-state literals assigned to two-state variables
1 participant