Skip to content
This repository has been archived by the owner on Mar 23, 2024. It is now read-only.

requireSpacesInsideArrayBrackets to ignore object literals #111

Closed
SBoudrias opened this issue Dec 27, 2013 · 4 comments
Closed

requireSpacesInsideArrayBrackets to ignore object literals #111

SBoudrias opened this issue Dec 27, 2013 · 4 comments

Comments

@SBoudrias
Copy link

Right now when setting requireSpacesInsideArrayBrackets, you can ignore on nested or require spaces everywhere.

It would be useful to be able to also ignore spaces if an object literal is provided, e.g.:

[{ foo: 'bar' }]
[{
  foo: 'bar'
}, {
  foo: 'bar'
}]

// but require
[ 'foo' ]
@markelog
Copy link
Member

markelog commented Feb 2, 2014

Relates to #92

@mikesherov
Copy link
Contributor

@markelog is this issue now handled?

@markelog
Copy link
Member

@mikesherov no :-(

@ameyms
Copy link

ameyms commented Aug 25, 2014

👍 for this enhancement (and also #92 )

@mikesherov mikesherov modified the milestones: 1.9, 1.10 Nov 28, 2014
markelog added a commit to markelog/node-jscs that referenced this issue Jan 15, 2015
Aside from main improvement:

* Simplify logic check

* Use assertion API

* Add couple more tests for main logic

* Fix accessor array case i.e. as expression statement

Fixes jscs-dev#111
Ref jscs-dev#721
Ref jscs-dev#429
markelog added a commit to markelog/node-jscs that referenced this issue Jan 15, 2015
Aside from main improvement:

* Simplify logic check

* Use assertion API

* Add couple more tests for main logic

* Fix accessor array case i.e. as expression statement

Fixes jscs-dev#111
Ref jscs-dev#721
Ref jscs-dev#429
markelog added a commit that referenced this issue Jan 15, 2015
Aside from main improvement:

* Simplify logic check

* Use assertion API

* Add couple more tests for main logic

* Fix accessor array case i.e. as expression statement

Fixes #111
Ref #722
Ref #429
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants