javascript/NativeScript
1
0
Fork 0
mirror of https://github.com/NativeScript/NativeScript.git synced 2025-08-16 11:42:04 +08:00
Code Issues Packages Projects Releases Wiki Activity

Validate package.json files' "main" setting.

Browse Source 
Don't use a filepath ending in .js, let the require implementation
resolve the js file. This makes it easier to require the correct
platform-specific file when bundling by switching the webpack
`resolve.extensions` setting.
This commit is contained in:
Hristo Deshev
2016-08-04 16:22:27 +03:00
parent 2e8458f274
commit 052f4617d5
1 changed files with 24 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

34
gruntfile.js
View File

@ -439,24 +439,37 @@ module.exports = function(grunt) {
"copy:childPackageFiles"
]);
grunt.registerTask("check-packagejson-boms", function() {
function hasBOM(filepath) {
var buf = grunt.file.read(filepath, { encoding: null });
return (buf[0] === 0xEF && buf[1] === 0xBB && buf[2] === 0xBF);
}
function validatePackageJsons(fileValidator, errorFormatter) {
var packageDescriptors = grunt.file.expand({}, [
'**/package.json',
'!node_modules/**'
'tns-core-modules/**/package.json'
]);
var errors = packageDescriptors.map(function(packagePath) {
if (hasBOM(packagePath)) {
return "File " + packagePath + " contains a UTF-8 BOM.";
if (fileValidator(packagePath)) {
return errorFormatter(packagePath);
} else {
return null;
}
}).filter(function(errorMessage) { return !!errorMessage; });
if (errors.length > 0)
grunt.fail.fatal("\n" + errors.join("\n"));
}
grunt.registerTask("check-packagejson-boms", function() {
validatePackageJsons(function (filepath) {
var buf = grunt.file.read(filepath, { encoding: null });
return (buf[0] === 0xEF && buf[1] === 0xBB && buf[2] === 0xBF);
}, function(filepath) {
return "File " + filepath + " contains a UTF-8 BOM.";
});
});
grunt.registerTask("check-packagejson-mains", function() {
validatePackageJsons(function (filepath) {
var packageData = grunt.file.readJSON(filepath);
return /\.js/i.test(packageData.main || "");
}, function(filepath) {
return "File " + filepath + " contains a broken main setting.";
});
});
grunt.registerTask("generate-tns-core-modules-dev-dts", generateModulesDts.bind(null, ".", localCfg.srcTnsCoreModules));
@ -475,6 +488,7 @@ module.exports = function(grunt) {
"clean:build",
"shell:getGitSHA",
"check-packagejson-boms",
"check-packagejson-mains",
"compile-ts",
"collect-modules-raw-files",
"copy:definitionFiles",
@ -557,4 +571,4 @@ module.exports = function(grunt) {
"pack-apps",
"get-ready-packages"
]));
};
};