rhiokim/grunt-sloc

Source line of codes plugin for Grunt.js

grunt-sloc's Issues

0.7.1 can't seem to get file filters to work using '.'

 sloc: {
                ui: {
                    options: {
                        reportDetail: true,
                    },
                    files: {
                        '.': ['catalogs/src/**/*.*']
                    }
                }
            }

always returns 0 files
however if I do

 files: {
'catalogs/src': ['**]
}

this works

what am i missing?

number of files read === 0

I tried to configure sloc and with --verbose. I can see a lot of selected files, but the number of files read is always zero.

			sloc: {
				'count': {
					options: {
						tolerant: true,
						reportDetail: true,
						reportType: "stdout"
					},
					files: {
						'.': ['<%= dir.src %>/**']

					}
				},
			},
...

Can anyone help me?

Counter not initialized with unsupported extension in tolerant mode

If I have a filter for '**.styl' files, your code fails because the counter for the fallback extension js has not been initialized.

You need to move the code initializing the counter one block down (to have it after the ext = 'js' piece of code).

Thanks, Dom

grunt sloc ignores files - Maybe side effect fix for #14

grunt sloc seems to be ignoring many files which are included in the src pattern.

To be more specific, here's my sloc config:

sloc: {
  'mylib': {
    files: {
      'mylib': ['*.js']
    }
  }
}

and here's my directory structure:

mylib/
mylib/one.js
mylib/two.js
mylib/util
mylib/util/three.js

number of files read by grunt sloc is zero. It should have been 2 files.

The problem does not occur after commenting out these lines related to solving issue #14:

    #14 negate match
     if (grunt.file.match(srcFilters, [f]).length === 0) {
        return;
     }

In the above code, f is an absolute path which seems to be the reason of the problem. One solution would have been to replace the above if condition with something like:

     var relPath = path.relative(path.join(process.cwd(), dest), f);
     if (grunt.file.match(srcFilters, [relPath ]).length === 0) {    

However after doing the above fix, src patterns such as **.js do not work.

I would think that instead of using readDir.readSync grunt's files.src.filter would do a better job. If you'd like to try this way, I can try to send a PR over the weekend.

HTML and SCSS support

Hey I added HTML and CSS support in sloc. Would be great to update grunt-sloc ;-)

flosse/sloc#5

require('ascii-table') but depends on 'asciitable'

https://github.com/rhiokim/grunt-sloc/blob/master/tasks/sloc.js#L16
var AsciiTable = require('ascii-table');
vs
https://github.com/rhiokim/grunt-sloc/blob/master/package.json#L34
"asciitable": "0.0.7"

Does not appear to be working

app
|-- Gruntfile.js
|-- notes.txt

[email protected]
grunt-sloc 0.4.2
grunt-cli v0.1.11
grunt v0.4.2
Mac OS X 10.9.1

'sloc': {
    'files': {
        './' : ['*.txt'],
    }
},

Upon cd app; grunt 'sloc', I only get:

Running "sloc:files" (sloc) task

Done, without errors.

Am I missing something?

Not working as intended

There is problem for me with line counting , I'm using OS X Mavericks 10.9.4, grunt-sloc 0.5.2.
And after running task through all files there're just zero's

Running "sloc:main" (sloc) task
Verifying property sloc.main exists in config...OK
Files: scripts/angular/application.js, scripts/angular/config/config.js, scripts/angular/controller/accounts.js, scripts/angular/controller/authentication.js, scripts/angular/controller/cards.js, scripts/angular/controller/controllers.js, scripts/angular/controller/footer.js, scripts/angular/controller/information.js, scripts/angular/controller/information_header.js, scripts/angular/controller/menu_navigation.js, scripts/angular/controller/payments.js, scripts/angular/directive/datePicker.js, scripts/angular/interceptor/responseError.js, scripts/angular/jettap.js, scripts/angular/router/routers.js, scripts/angular/service/authentication.js, scripts/angular/service/services.js, scripts/angular/templates/templates.js -> src
Options: reportType="stdout", reportPath=null, reportDetail, tolerant=false

  physical lines         0       
  lines of source code   0       
  total comment          0       
  singleline             0                 
  multiline              0                 
  empty                  0       

  number of files read   0       
  mode                   strict  

.
 |
-|
'

Done, without errors.
Process finished with exit code 0

Doesn't Work With "watch"

Using watch then changing a file, sloc doesn't output the results again.

Cannot benefit from grunt negate matches

I would like to be able to exclude files from the directories to be parsed. For example, I would like to specify:

css: {
   'src/client': [ '**/**.styl', '!**/libs/**' ]
}

You could support them with the following lines added before the call to sloc(source, ext):

if (grunt.file.match(srcFilters, [f]).length === 0) {
   return;
}

Thanks, Dom

Support LESS

I was looking through the code. I see it supports SASS. Is there any change besides adding an entry for less in the array to support LESS? If this is the case, can I submit a PR for it?

support extension

Grunt issues warning Warning: Object -1 has no method 'replace' Use --force to continue.

I am seeing a warning being thrown by grunt after updating the fix to 0.4.1.

Running "sloc:scripts" (sloc) task
Warning: Object -1 has no method 'replace' Use --force to continue.

Fails CI for me!

Not working as intended

Directory:

app
|-- gruntfile.js
|-- server.js

In gruntfile.js:

sloc: {
        files: {
            './' : [ 'server.js' ],
                    }
     },

Terminal:

grunt sloc
Running "sloc:files" (sloc) task

Done, without errors.

? Windows.

Multiple Tasks Combine Results Instead of Staying Separate

I am using the following, and the output combines the first result (server) into the second report (client), instead of separating the two results.

sloc: {
    server: {
        files: {
            'server': ['**/*.js'],
            './': ['Gruntfile.js', 'server.js']
        }
    },
    client: {
        files: {
            'client': ['index.html']
        }
    }
},

OUTPUT:

Running "sloc:server" (sloc) task

physical lines : 221
lines of source code : 152
total comment : 47
singleline : 47
multiline : 0
empty : 22

number of files read : 2

---

Running "sloc:client" (sloc) task

physical lines : 379
lines of source code : 295
total comment : 47
singleline : 47
multiline : 0
empty : 37

number of files read : 3

"File extension 'html' is not supported"

sloc: {
    client: {
        files: {
            'client': ['index.html', 'robots.txt', 'js/app/**','views/**']
        }
    }
}

All 0s in generated JSON file

When I run the following command:

                jformDistJson: {
                    options: {
                        reportType: 'json',
                        reportPath: '../doc/sloc/jform-dist-sloc-<%= pkg.version %>.json',
                    },
                    files: {
                        '../js': ['jForm/*.js']
                    }
                }

It produces this output:

Running "sloc:jformDistJson" (sloc) task
Create at: ../doc/sloc/jform-dist-sloc-1.7.1.json

Done, without errors.

I get all 0s in the JSON output:

{
  "total": 0,
  "source": 0,
  "comment": 0,
  "single": 0,
  "mixed": 0,
  "empty": 0,
  "block": 0,
  "file": 0
}

I am using:
Windows 2008 R2
grunt 0.4.5
grunt-sloc 0.7.1
sloc 0.1.10
node.js 4.4.0

This same relative path works in all of my other grunt tasks. In fact, I've tried seemingly every combination of absolute and relative paths and I get the same result. Can you please help?