Skip to content
This repository has been archived by the owner on Dec 29, 2020. It is now read-only.

Correctly handle folders. #65

Closed
wants to merge 1 commit into from

Conversation

arthurschreiber
Copy link

With a Gruntfile like this:

    jscs: {
      files: [ '.' ],
      options: {
        config: true,
      }
    }

Only style issues for the first file in the folder get reported. This fixes this issue so that style issues for all files inside the . folder get reported.

With a Gruntfile like this:

```js
    jscs: {
      files: [ '.' ],
      options: {
        config: true,
      }
    }
```

Only style issues for the first file in the folder get reported. This fixes this issue so that style issues for all files inside the `.` folder get reported.
@markelog markelog closed this in a8c09f0 Nov 15, 2014
@markelog
Copy link
Member

Not a fan of expressing everything with Array#reduce and there was no tests and there was code-style violations.

So i hope you don't mind i fixed it myself.

Thank you for reporting this issue though.

@arthurschreiber
Copy link
Author

Sure, no worries. The important thing is that the issue was fixed. 👍 Thanks! ❤️

@gustavohenke
Copy link
Member

@arthurschreiber in the next few days a new version of JSCS will be released so I'll bump grunt-jscs version as well with this fix

@markelog
Copy link
Member

in the next few days a new version of JSCS will be released so I'll bump grunt-jscs version as well with this fix

50/50 :-(

@gustavohenke
Copy link
Member

Released in v1.0.0!

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.

3 participants