Skip to content

Can we enable the dangling comma on this project? #3469

@lipis

Description

@lipis

Because even though one line was added, two were deleted and that's confusing when reviewing.

screen shot 2017-12-13 at 11 39 30

Also if you would like to sort them, you would have to manually add the missing comma, because the there will be a syntax error if the last item will move in the middle.. (if someone will mention that these things are handled by Prettier 😃)

More reasons can be found here: https://medium.com/@nikgraf/why-you-should-enforce-dangling-commas-for-multiline-statements-d034c98e36f8

Metadata

Metadata

Assignees

No one assigned

    Labels

    locked-due-to-inactivityPlease open a new issue and fill out the template instead of commenting.status:has prIssues with an accompanying pull request. These issues will probably be fixed soon!status:needs discussionIssues needing discussion and a decision to be made before action can be takentype:infraIssues about CI, publishing to npm, or similar

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions