lourenzo / gulp-jshint-xml-file-reporter Goto Github PK

View Code? Open in Web Editor NEW

4.0 4.0 6.0 386 KB

A JSHint reporter to be used by gulp-jshint that will provide a jslint.xml that can be used by CI tools as jenkins

License: MIT License

JavaScript 100.00%

gulp-jshint-xml-file-reporter's People

Contributors

Stargazers

Watchers

Forkers

shnist cleydsonjr smaggs gregoriol jorisvddonk 7oi

gulp-jshint-xml-file-reporter's Issues

Clean run does not create report file

It would be nice if the reporter always produced a report file and not only if errors occurred. Currently once I correct a jshint warning my report file is not updated to indicate 0 errors and instead will contain the errors prior to the successful run.

If checkstyle file directory doesn't exist the reporter fails

It will throw an error like this:

events.js:72
        throw er; // Unhandled 'error' event
              ^
Error: ENOENT, open 'test_out/jshint.xml'

if the target folder is not existing, the file can't be written

If you have a clean task on your output folder, which removes all content and subfolders and then jshint is the first task which is running, then it can't write the output file because of a missing folder structure.

Workaround:

gulp.task('prepareStylecheck', function () {
    //just create the output folder because of a reporter bug
    return gulp.src(PATH_IN_SRC)
        .pipe(gulp.dest(PATH_OUT_TESTRESULT));
});

gulp.task('stylecheck', ['prepareStylecheck'], function () {
    return gulp.src(FILES_SRC)
        .pipe(jshint())
        .pipe(jshint.reporter('jshint-stylish'))
        .pipe(jshint.reporter(jshintXMLReporter))
        .on('end', jshintXMLReporter.writeFile({
            format: 'checkstyle',
            filePath: PATH_OUT_TESTRESULT + 'checkstyle.xml'
        }));
});

Create a new release

Hi,

I would be very grateful if you could create a new release through npm version and create a new tag for the new updates that have been pulled into the repository recently.

Many thanks

Creating a new release for 0.4

Hi,

I would be very grateful if you could create a new release for version 0.4

Thanks :)

Consider using fs-extra#createOutputStream instead of fs#createWriteStream

If the filePath specifies a directory path that does not exist, createWriteStream fails with

Error: ENOENT: no such file or directory, open 'path/to/jshint/results.xml'

createOutputStream from the fs-extra package handles this correctly.

jshint_xml formatter: Cannot find module './lib/jshint_xml_emitter'

Hi,

I tried the plugin for the first time today, and i have an issue when i try to use the jshint-xml formatter instead of the default one (checkstyle).

Here are some inputs:

The gulp task

   return gulp.src('app/scripts/**/*.js')
        .pipe($.jshint())
        .pipe($.jshint.reporter($.jshintXmlFileReporter))
        .on('end', $.jshintXmlFileReporter.writeFile({
            format: 'jshint_xml',
            filePath: './jshint.xml'
        }));

A dump of the stack when the gulp task is called :

[14:13:18] Error: Cannot find module './lib/jshint_xml_emitter'
    at Function.Module._resolveFilename (module.js:338:15)
    at Function.Module._load (module.js:280:25)
    at Module.require (module.js:364:17)
    at require (module.js:380:17)
    at loadFormatter (/home/spiroid/test/node_modules/gulp-jshint-xml-file-reporter/index.js:17:12)
    at Object.exports.writeFile (/home/spiroid/test/node_modules/gulp-jshint-xml-file-reporter/index.js:34:26)
    at Gulp.<anonymous> (/home/spiroid/test/gulp/lint.js:19:44)
    at module.exports (/home/spiroid/test/node_modules/gulp/node_modules/orchestrator/lib/runTask.js:34:7)
    at Gulp.Orchestrator._runTask (/home/spiroid/test/node_modules/gulp/node_modules/orchestrator/index.js:273:3)
    at Gulp.Orchestrator._runStep (/home/spiroid/test/node_modules/gulp/node_modules/orchestrator/index.js:214:10)

The jslint_xml_emitter.js is located in the project node_modules/gulp-jshint-xml-file-reporter/lib directory
plugin version : 0.3.3

Did i forget something in my gulp task / configuration ?
Thanks.

lourenzo / gulp-jshint-xml-file-reporter Goto Github PK

gulp-jshint-xml-file-reporter's People

Contributors

Stargazers

Watchers

Forkers

gulp-jshint-xml-file-reporter's Issues

Clean run does not create report file

If checkstyle file directory doesn't exist the reporter fails

if the target folder is not existing, the file can't be written

Create a new release

Creating a new release for 0.4

Consider using fs-extra#createOutputStream instead of fs#createWriteStream

jshint_xml formatter: Cannot find module './lib/jshint_xml_emitter'

Recommend Projects

React

Vue.js

Typescript

TensorFlow

Django

Laravel

D3

Recommend Topics

javascript

web

server

Machine learning

Visualization

Game

Recommend Org

Facebook

Microsoft

Google

Alibaba

D3

Tencent