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

Additional async rules #45

Merged
merged 3 commits into from Oct 10, 2017

Conversation

@alexashley
Copy link
Contributor

commented Oct 5, 2017

  • require-await prevents any unnecessary async functions (maybe the result of a refactoring)
  • no-return-await has gives a slight performance gain and also prevents unnecessary async functions
alexashley added 3 commits Oct 5, 2017
@alexashley alexashley changed the title Require async functions to await Additional async rules Oct 5, 2017
@manovotny

This comment has been minimized.

Copy link
Owner

commented Oct 10, 2017

This will fix #41.

@manovotny manovotny changed the base branch from master to v3 Oct 10, 2017
@manovotny

This comment has been minimized.

Copy link
Owner

commented Oct 10, 2017

Merging this into the v3 branch, instead of master, since this is a breaking change.

@manovotny manovotny merged commit 9b3b971 into manovotny:v3 Oct 10, 2017
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.