openstack/eslint-config-openstack
Code Issues Proposed changes

Stop using IIFE for index.js

Browse Source 
`index.js` is a NodeJS file. NodeJS automatically adds IIFE around
JS code from each NodeJS file and passes in some variables like
`module` object.  Therefore it is redundant to have IIFE in
`index.js`.

Change-Id: Iea8ad83d8f3598aa506063614db482eeef52db4e
This commit is contained in:
Shaoquan Chen 2015-09-01 19:30:21 -07:00 committed by Michael Krotscheck
parent e31eb6dcd9
commit f0d0b3fc81
1 changed files with 12 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
index.js
View File

@ -14,21 +14,22 @@
* under the License. * under the License.
*/ */
/*eslint-disable strict*/
'use strict';
/*eslint-enable strict*/
/** /**
* The default module for this package simply reads in the .eslintrc yaml file, and returns it * The default module for this package simply reads in the .eslintrc yaml file, and returns it
* as a module. * as a module.
*/ */
(function () {
"use strict";
var yaml = require('js-yaml'); var yaml = require('js-yaml');
var fs = require('fs'); var fs = require('fs');
var path = require('path'); var path = require('path');
/*eslint-disable no-sync */ /*eslint-disable no-sync */
var rcPath = path.join(__dirname, '.eslintrc'); var rcPath = path.join(__dirname, '.eslintrc');
var fileContent = fs.readFileSync(rcPath); var fileContent = fs.readFileSync(rcPath);
/*eslint-enable no-sync */ /*eslint-enable no-sync */
module.exports = yaml.safeLoad(fileContent); module.exports = yaml.safeLoad(fileContent);
})();