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

Completed the assignment! #16

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

Completed the assignment! #16

wants to merge 11 commits into from

Conversation

khintk
Copy link

@khintk khintk commented Mar 12, 2018

Sorry for my late pull request! I didn't know that I was supposed to make a pull request.

Khin Thiri Kyaw and others added 11 commits February 5, 2018 16:50
…class_structure,test_static_types, test_type_relationship have passed
Worked on checking if the method name exist, and if the parameters are right
Worked on implementing the type error messages for the 1.3.
fixed the check_types() for ConstructorCall()! There are five types that are not passing regarding the types of arguemnts passed in. Tried solving but I cant seem to find where I got it wrong!
In check_types() of method, instead of raising the JavaTypeError, I was just rasing the TypeError. Fixed it, and a few more tests are passing
Fixed the same typo in rasing JavaTypeError in Constructor Call
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant