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

Remove deprecated stuff #92

Merged
merged 1 commit into from
Nov 28, 2014
Merged

Remove deprecated stuff #92

merged 1 commit into from
Nov 28, 2014

Conversation

tarmolov
Copy link
Member

No description provided.

@coveralls
Copy link

Coverage Status

Coverage increased (+1.05%) when pulling f3dea7d on hevil.deprecated into 2c66b35 on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+1.05%) when pulling 0b347a5 on hevil.deprecated into 2c66b35 on master.

@@ -93,25 +75,15 @@
* @param {Object} params Data should be sent to the method.
* @param {Object} [execOptions] Exec-specific options.
* @param {Boolean} [execOptions.enableBatching=true] Should the current call of the method be batched.
* @param {Boolean} [execOptions.noBatching=false] @deprecated Should the current call of the
* method be batched.

Choose a reason for hiding this comment

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

Let's remove this line, it's a part of the description above.

@watch-the-stars
Copy link

👍

noBatching: !isArray(options.noBatching) && Boolean(options.noBatching)
enableBatching: options.hasOwnProperty('enableBatching') ?
options.enableBatching :
true
Copy link
Contributor

Choose a reason for hiding this comment

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

It would be good to add a comment that batching is enabled by default and that's why we have a boolean literal here.

Copy link
Member Author

Choose a reason for hiding this comment

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

See jsdoc

@dodev
Copy link
Contributor

dodev commented Nov 28, 2014

Purification 👍

tarmolov added a commit that referenced this pull request Nov 28, 2014
@tarmolov tarmolov merged commit 102d0ee into master Nov 28, 2014
@tarmolov tarmolov deleted the hevil.deprecated branch November 28, 2014 21:39
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.

4 participants