-
Notifications
You must be signed in to change notification settings - Fork 38
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
Handling raised MatchExceptions in the Determiner Object and adding more adjective and adverb support #290
Conversation
Merge changes in original
Signed-off-by: sundararajan-s <[email protected]>
Signed-off-by: sundararajan-s <[email protected]>
# Conflicts: # docs/index.html
Signed-off-by: sundararajan-s <[email protected]>
# Conflicts: # docs/index.html
Signed-off-by: sundararajan-s <[email protected]> # Conflicts: # docs/index.html
The diffs are too large so it is hard to tell what actually changed. For example, why should |
A couple of points:
|
Signed-off-by: sundararajan-s <[email protected]> # Conflicts: # docs/index.html
I think this was due to reformatting. I shall fix that.
I shall do that. |
I shall redo this and create a different pull request. |
Handles the issue regarding of a
MatchException
being raised by the Determiner Object by adding new cases and a new subclass for the handling 'both' #289.