lcompilers / lpython

Python compiler
https://lpython.org/
Other
1.5k stars 158 forks source link

add test reference for an error message #2623

Closed aryangupta701 closed 6 months ago

aryangupta701 commented 6 months ago

fixes #1953

add a test reference in order to address #1953 issue

aryangupta701 commented 6 months ago

any suggestions for the test file's name?

kmr-srbh commented 6 months ago

any suggestions for the test file's name?

Hello @aryangupta701! I think the file can be renamed to say test_optional.py since we are testing for the Optional type, which is a part of LFortran, but not supported here in LPython.

kmr-srbh commented 6 months ago

I request you to link this PR with the issue so that it gets closed automatically when this is merged. You can add fixes #1953 in the PR description to link it.