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

Add error msg for type checking #1211

Closed
wants to merge 39 commits into from

Conversation

tiashah1
Copy link

Making the input take "runParser" arguments
Adding error messages instead of returning false or Nothing
Adding some more expressions (e.g. Lambda)

@tiashah1
Copy link
Author

@dorchard FYI
Another draft PR - this time focusing on error messages (and refactoring check and synth to allow us to use (runParser ...) statements as inputs

@dorchard dorchard changed the title Add error msg Add error msg for type checking Jan 13, 2025
@dorchard dorchard added the types label Jan 13, 2025
@tiashah1
Copy link
Author

  • Error messages are a String as we do not have other information currently. New file TypeCheckError
  • Replaced the check and synth functions to allow for: Either a (Expr b)
    • This needs to be amended as incorrect as is
  • Added check and synth patterns for
    • Lambda
    • App
    • MatchAs
    • DProject (only for synth)
    • Matrix (only for check)
  • Altered the Dictionary patterns in check and synth to line up with changes from develop (removing Records, etc.)
    • Added a new type: TDict Types Types

@tiashah1
Copy link
Author

closing this PR and re-opening, instead pointing to the functions-and-integers branch in my own fork
tiashah1#1

@tiashah1 tiashah1 closed this Jan 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants