Skip to content

Conversation

@graysonarts
Copy link
Contributor

parameter_added_in got a facelift and now takes keyword arguments, where the value represents the version for that keyword. It looks a little cleaner and makes the checking code much simpler.

Also renamed the extract_only to be no_extract because it means the opposite of extract_only, oops.

(cherry-picking this commit to fix the rebase error)

`parameter_added_in` got a facelift and now takes keyword arguments, where the value represents the version for that keyword. It looks a little cleaner and makes the checking code much simpler.

Also renamed the `extract_only` to be `no_extract` because it means the opposite of `extract_only`, oops.
Copy link
Collaborator

@t8y8 t8y8 left a comment

Choose a reason for hiding this comment

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

🚀

@graysonarts graysonarts merged commit 470d2ef into tableau:development Apr 14, 2017
@graysonarts graysonarts deleted the fix-parameter_added_in branch April 14, 2017 18:02
t8y8 pushed a commit to t8y8/server-client-python that referenced this pull request Jun 28, 2017
…leau#175)

`parameter_added_in` got a facelift and now takes keyword arguments, where the value represents the version for that keyword. It looks a little cleaner and makes the checking code much simpler.

Also renamed the `extract_only` to be `no_extract` because it means the opposite of `extract_only`, oops.
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