Join GitHub today
GitHub is home to over 40 million developers working together to host and review code, manage projects, and build software together.
Sign upNew rule: object-property-newline #524
Comments
feross
added
the
enhancement
label
May 17, 2016
feross
closed this
in
standard/eslint-config-standard@ebe08d0
Jul 13, 2016
This comment has been minimized.
This comment has been minimized.
|
This rule will be included in standard v8 |
This comment has been minimized.
This comment has been minimized.
KidkArolis
commented
Sep 3, 2016
|
Hm, this means I can no longer use a "dsl" like this :( module.exports = [
{ name: 'application', path: '/', component: Application, abstract: true, children: [
{ name: 'home', path: '', component: Home },
{ name: 'messages', async: Messages },
{ name: 'status', path: ':user/status/:id' },
{ name: 'profile', path: ':user', component: Profile, abstract: true, children: [
{ name: 'profile.index', path: '', component: ProfileIndex },
{ name: 'profile.lists' },
{ name: 'profile.edit' }
]}
]}
] |
This comment has been minimized.
This comment has been minimized.
|
@KidkArolis Sorry this makes life a bit more difficult for you. You can selectively disable this rule in files where you're defining DSLs by adding a comment to the top of the file: /* eslint-disable object-property-newline */ |
This comment has been minimized.
This comment has been minimized.
KidkArolis
commented
Sep 10, 2016
|
Yeah, no worries, <3 standard, found a couple workarounds.
|
feross commentedMay 17, 2016
Enforce placing object properties on separate lines.
eslint rule: https://github.com/eslint/eslint/blob/master/docs/rules/object-property-newline.md
OK:
Not OK:
Ecosystem breakage: