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

Implement Type Error #25

Closed
lucas-oma opened this issue May 29, 2021 · 3 comments · Fixed by #36, #38 or #47
Closed

Implement Type Error #25

lucas-oma opened this issue May 29, 2021 · 3 comments · Fixed by #36, #38 or #47
Assignees
Labels
difficulty/medium enhancement New feature or request help wanted Extra attention is needed

Comments

@lucas-oma
Copy link
Owner

Implement Type Error

Description:
An error message must be printed when a type error occurs.
For now we will implement Panic-Mode Error Recovery. This can be upgraded in the future.

@lucas-oma lucas-oma added enhancement New feature or request help wanted Extra attention is needed labels May 29, 2021
@lucas-oma lucas-oma added this to the Alpha release milestone May 29, 2021
@lucas-oma lucas-oma self-assigned this Jun 1, 2021
@shreegilliorkar
Copy link
Contributor

@martinezlucas98 , may i look into this issue. Can you please tell more detail about it like where the error is detected and which files to be refereed to solve this issue.

@lucas-oma
Copy link
Owner Author

lucas-oma commented Jun 5, 2021

@Shree-Gillorkar right now I am looking at and testing a PR that someone made for issue #26 which I think is necessary before we can attemp to solve this one (although I could be wrong).

Anyway, the idea here is to do type conversions (and implicit conversion) and check type compatibility when declaring a variable -right and left sides - (also an implicit conversion, I think)

But I will let you know as soon as the PR I mentioned is merged and if this issue has not been resolved in that same PR.
Anyway, thanks for your interest!

@shreegilliorkar
Copy link
Contributor

@martinezlucas98 ya its all okay. Let me know whenever you complete that PR you talked about.

@lucas-oma lucas-oma linked a pull request Jun 14, 2021 that will close this issue
lucas-oma added a commit that referenced this issue Jun 14, 2021
Merge pull request #36 from martinezlucas98/owner
@lucas-oma lucas-oma linked a pull request Jun 15, 2021 that will close this issue
lucas-oma added a commit that referenced this issue Jun 15, 2021
Merge pull request #38 from martinezlucas98/owner
@lucas-oma lucas-oma linked a pull request Jun 17, 2021 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
difficulty/medium enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
2 participants