We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Tested by changing line 147 of chipper/grunt/Gruntfile.js to:
repoFiles: [ 'js//*.js', 'requirejs-plugins//*.js' ],
then running grunt lint.
grunt lint
Linting requirejs-plugins/loadFileAsDataURI.js ...ERROR [L27:C52] W064: Missing 'new' prefix when invoking a constructor. var base64 = 'data:' + mimeType + ';base64,' + Buffer( fs.readFileSync( filename ) ).toString( 'base64' ); Linting requirejs-plugins/loadFileAsDataURI.js ...ERROR [L27:C52] W117: 'Buffer' is not defined. var base64 = 'data:' + mimeType + ';base64,' + Buffer( fs.readFileSync( filename ) ).toString( 'base64' ); Linting requirejs-plugins/loadFileAsDataURI.js ...ERROR [L27:C60] W117: 'fs' is not defined. var base64 = 'data:' + mimeType + ';base64,' + Buffer( fs.readFileSync( filename ) ).toString( 'base64' );
The text was updated successfully, but these errors were encountered:
#96 lint fixes for loadFileAsDataURI.js
aed984c
Fixed and tested various lint issues. And according to https://docs.nodejitsu.com/articles/advanced/buffers/how-to-use-buffers, it should be "new Buffer".
Closing, feel free to review if anyone is inclined.
Sorry, something went wrong.
f6b8d05
pixelzoom
No branches or pull requests
Tested by changing line 147 of chipper/grunt/Gruntfile.js to:
then running
grunt lint
.The text was updated successfully, but these errors were encountered: