Skip to content

Feature ext#226

Merged
machow merged 3 commits intomasterfrom
feature-ext
Jun 21, 2017
Merged

Feature ext#226
machow merged 3 commits intomasterfrom
feature-ext

Conversation

@machow
Copy link
Copy Markdown
Contributor

@machow machow commented Jun 21, 2017

Implements features used in sqlwhat necessary for pythonwhat-ext!

@machow machow mentioned this pull request Jun 21, 2017
@machow machow requested a review from filipsch June 21, 2017 16:15
@machow machow self-assigned this Jun 21, 2017
Comment thread pythonwhat/check_syntax.py Outdated
elif type(f) == Chain:
raise BaseException("did you use a result of the Ex() function on the right hand side of the + operator?")
elif not callable(f):
raise BaseException("right hand side of + operator should be an SCT, so must be callable!")
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

of >> operator?

Comment thread pythonwhat/check_syntax.py Outdated
if self._waiting_on_call:
self._double_attr_error()
elif type(f) == Chain:
raise BaseException("did you use a result of the Ex() function on the right hand side of the + operator?")
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

of the >> operator?

Copy link
Copy Markdown
Contributor

@filipsch filipsch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe some typos in the messages; other than that all good!

@machow machow merged commit 22b8fec into master Jun 21, 2017
@machow machow deleted the feature-ext branch January 2, 2018 15:33
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.

2 participants