Skip to content
This repository has been archived by the owner on Oct 2, 2021. It is now read-only.

feat(rule): add blaze-consistent-eventmap-parameters #33

Closed
wants to merge 8 commits into from

Conversation

PhilippSpo
Copy link

Add rule that enforces consistent event handler parameters.

closes #12


CallExpression(node) {

// const config = context.options[0]
Copy link
Contributor

Choose a reason for hiding this comment

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

Remove this?

Copy link
Author

Choose a reason for hiding this comment

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

done


module.exports = getMeta => context => {

const {env} = getMeta(context.getFilename())
Copy link
Contributor

Choose a reason for hiding this comment

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

const {env, isLintedEnv} = getMeta(..)

ensureParamName does not get the whole node with all the params, but only the param that it needs
dferber90 added a commit that referenced this pull request Oct 28, 2015
@dferber90
Copy link
Contributor

Closed through be73ffa. Thanks @PhilippSpo 😊

@dferber90 dferber90 closed this Oct 28, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Rule Proposal: blaze-consistent-eventmap-params
2 participants