no-return-wrap crash eslint on empty return statement

[zloirock]

promise.then(it => {
    return;
});

=>

Cannot read property 'type' of null
TypeError: Cannot read property 'type' of null
    at EventEmitter.ReturnStatement (/Users/zloirock/wd/static/node_modules/eslint-plugin-promise/rules/no-return-wrap.js:25:28)
    at emitOne (events.js:101:20)
    at EventEmitter.emit (events.js:188:7)
    at NodeEventGenerator.enterNode (/Users/zloirock/wd/static/node_modules/eslint/lib/util/node-event-generator.js:40:22)
    at CodePathAnalyzer.enterNode (/Users/zloirock/wd/static/node_modules/eslint/lib/code-path-analysis/code-path-analyzer.js:608:23)
    at CommentEventGenerator.enterNode (/Users/zloirock/wd/static/node_modules/eslint/lib/util/comment-event-generator.js:97:23)
    at Controller.enter (/Users/zloirock/wd/static/node_modules/eslint/lib/eslint.js:925:36)
    at Controller.__execute (/Users/zloirock/wd/static/node_modules/eslint/node_modules/estraverse/estraverse.js:397:31)
    at Controller.traverse (/Users/zloirock/wd/static/node_modules/eslint/node_modules/estraverse/estraverse.js:501:28)
    at Controller.Traverser.controller.traverse (/Users/zloirock/wd/static/node_modules/eslint/lib/util/traverser.js:36:33)

Latest versions of eslint-plugin-promise, eslint, babel-eslint

[xjamundx]

@zloirock can you take a stab at this?

[bonesoul]

it's still crashing eslint for me. is this resolved?

[leedm777]

@bonesoul I can confirm that the fix works for me, but it hasn't been published to NPM yet.

[xjamundx]

let me see what's up

…

On Mon, Feb 6, 2017 at 2:16 PM David M. Lee ***@***.***> wrote: @bonesoul <https://github.com/bonesoul> I can confirm that the fix works for me, but it hasn't been published to NPM yet. — You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub <#49 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAPBf9sNqZhsm-19a8AG9OEIN95K8tFzks5rZ5uzgaJpZM4K8gI1> .

[xjamundx]

published `v3.4.1`

…

On Mon, Feb 6, 2017 at 2:16 PM Jamund Ferguson ***@***.***> wrote: let me see what's up On Mon, Feb 6, 2017 at 2:16 PM David M. Lee ***@***.***> wrote: @bonesoul <https://github.com/bonesoul> I can confirm that the fix works for me, but it hasn't been published to NPM yet. — You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub <#49 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAPBf9sNqZhsm-19a8AG9OEIN95K8tFzks5rZ5uzgaJpZM4K8gI1> .

[leedm777]

@xjamundx That's some awesome response time. Thanks!