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

New: group-exports rule #898

Closed
wants to merge 1 commit into from
Closed

New: group-exports rule #898

wants to merge 1 commit into from

Conversation

robertrossmann
Copy link
Contributor

This adds a new rule, group-exports.

Reports when named exports are not grouped together in a single export declaration or when multiple assignments to CommonJS module.exports or exports object are present in a single file.

Rationale: An export declaration or module.exports assignment can appear anywhere in the code. By requiring a single export declaration all your exports will remain at one place, making it easier to see what exports a module provides.

For extensive discussion, see #721.

@ljharb
Copy link
Member

ljharb commented Jul 19, 2017

Closing as a duplicate of #721; please reopen that one.

@ljharb ljharb marked this as a duplicate of #721 Jul 19, 2017
@ljharb ljharb closed this Jul 19, 2017
@robertrossmann robertrossmann deleted the group-exports branch July 19, 2017 18:41
@coveralls
Copy link

coveralls commented Jul 19, 2017

Coverage Status

Coverage increased (+0.06%) to 96.022% when pulling d230d5e on robertrossmann:group-exports into ccd9394 on benmosher:master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants