{"action":"created","issue":{"url":"https://api.github.com/repos/kenzanmedia/pipeline-validate-js/issues/17","labels_url":"https://api.github.com/repos/kenzanmedia/pipeline-validate-js/issues/17/labels{/name}","comments_url":"https://api.github.com/repos/kenzanmedia/pipeline-validate-js/issues/17/comments","events_url":"https://api.github.com/repos/kenzanmedia/pipeline-validate-js/issues/17/events","html_url":"https://github.com/kenzanmedia/pipeline-validate-js/issues/17","id":112263964,"number":17,"title":"validate functions returning unexpected linting errors","user":{"login":"thescientist13","id":895923,"avatar_url":"https://avatars.githubusercontent.com/u/895923?v=3","gravatar_id":"","url":"https://api.github.com/users/thescientist13","html_url":"https://github.com/thescientist13","followers_url":"https://api.github.com/users/thescientist13/followers","following_url":"https://api.github.com/users/thescientist13/following{/other_user}","gists_url":"https://api.github.com/users/thescientist13/gists{/gist_id}","starred_url":"https://api.github.com/users/thescientist13/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/thescientist13/subscriptions","organizations_url":"https://api.github.com/users/thescientist13/orgs","repos_url":"https://api.github.com/users/thescientist13/repos","events_url":"https://api.github.com/users/thescientist13/events{/privacy}","received_events_url":"https://api.github.com/users/thescientist13/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-10-20T01:17:26Z","updated_at":"2015-10-20T02:02:56Z","closed_at":null,"body":"Given a directory structure\r\n```\r\nnode_modules/\r\npipelines/javascript.js\r\n.jshintrc\r\n.jscsrc\r\ngulpfile.js\r\n```\r\n\r\nand this linting task\r\n```javascript\r\nvar loadPluginsConfig = require('./configs/load-plugin-config.json');\r\nvar $ = require('gulp-load-plugins')(loadPluginsConfig);\r\nvar gulp = require('gulp-param')(require('gulp'), process.argv);\r\n\r\ngulp.task('js:lint', function () {\r\n\r\n  return gulp.src(['*.js', './pipelines/*', './src/**/**/*.js'])\r\n    .pipe($.jshint())\r\n    .pipe($.jscs())\r\n    .pipe($.jscsStylish.combineWithHintResults())\r\n    .pipe($.jshint.reporter('jshint-stylish'))\r\n    .pipe($.jshint.reporter('fail', { verbose: true }));\r\n\r\n});\r\n```\r\n\r\nwhen running `js:lint` the reporter reports success\r\n```bash\r\n$ gulp js:lint\r\n[21:10:14] Using gulpfile ~/Workspace/analogstudios.net/repo/as-webapp-2.0/gulpfile.js\r\n[21:10:14] Starting 'js:lint'...\r\n[21:10:15] Finished 'js:lint' after 743 ms\r\n```\r\n\r\nHowever, when converting to use `pipeline-validate-js`\r\n```javascript\r\nvar validatePipeline = require('pipeline-validate-js')();\r\nvar gulp = require('gulp-param')(require('gulp'), process.argv);\r\n\r\ngulp.task('js:lint', function () {\r\n\r\n  return gulp.src(['*.js', './pipelines/*', './src/**/**/*.js'])\r\n    .pipe(validatePipeline.validateJS());\r\n});\r\n```\r\n\r\nI get jshint and jscs reporting errors.\r\n\r\nThe pipeline seems to be getting the path to my config files correctly (and falling back to bundled config when not present)\r\n```bash\r\n$ gulp js:lint\r\nfilename => .jshintrc\r\nprocess cwd => /Users/obuckley/Workspace/analogstudios.net/repo/as-webapp-2.0\r\nresolveConfigFile => /Users/obuckley/Workspace/analogstudios.net/repo/as-webapp-2.0/.jshintrc\r\n\r\nfilename => .jscsrc\r\nprocess cwd => /Users/obuckley/Workspace/analogstudios.net/repo/as-webapp-2.0\r\nresolveConfigFile => /Users/obuckley/Workspace/analogstudios.net/repo/as-webapp-2.0/.jscsrc\r\n\r\nfilename => .eslintrc\r\nprocess cwd => /Users/obuckley/Workspace/analogstudios.net/repo/as-webapp-2.0\r\nresolveConfigFile => /Users/obuckley/Workspace/analogstudios.net/repo/as-webapp-2.0/node_modules/pipeline-validate-js/.eslintrc\r\n```"},"comment":{"url":"https://api.github.com/repos/kenzanmedia/pipeline-validate-js/issues/comments/149401268","html_url":"https://github.com/kenzanmedia/pipeline-validate-js/issues/17#issuecomment-149401268","issue_url":"https://api.github.com/repos/kenzanmedia/pipeline-validate-js/issues/17","id":149401268,"user":{"login":"thescientist13","id":895923,"avatar_url":"https://avatars.githubusercontent.com/u/895923?v=3","gravatar_id":"","url":"https://api.github.com/users/thescientist13","html_url":"https://github.com/thescientist13","followers_url":"https://api.github.com/users/thescientist13/followers","following_url":"https://api.github.com/users/thescientist13/following{/other_user}","gists_url":"https://api.github.com/users/thescientist13/gists{/gist_id}","starred_url":"https://api.github.com/users/thescientist13/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/thescientist13/subscriptions","organizations_url":"https://api.github.com/users/thescientist13/orgs","repos_url":"https://api.github.com/users/thescientist13/repos","events_url":"https://api.github.com/users/thescientist13/events{/privacy}","received_events_url":"https://api.github.com/users/thescientist13/received_events","type":"User","site_admin":false},"created_at":"2015-10-20T02:02:56Z","updated_at":"2015-10-20T02:02:56Z","body":"Looks like it's only JSCS that is throwing errors.  JSHint config resolves fine, I tested by setting `quotes: double` in my jshint and `quotes: single` inside index.js, and got \"Strings must use doublequote.\"  \r\n\r\nThis means that my personal config file is superseding the bundled one (which is expected)"}}