Merge pull request #261 from Microsoft/npm_checks

Exclude NPM checks from check-parse-results when run as part of a test/publish

Author: Andy

packages/types-publisher/src/check-parse-results.ts

@@ -6,18 +6,20 @@ import { fetchJson} from "./util/io";
 import { best, done, nAtATime } from "./util/util";
 
 if (!module.parent) {
-	done(main());
+	done(main(true));
 }
 
-export default async function main(): Promise<void> {
+export default async function main(includeNpmChecks: boolean): Promise<void> {
 	const packages = await AllPackages.readTypings();
 	const [log, logResult] = logger();
 	check(packages, info => info.libraryName, "Library Name", log);
 	check(packages, info => info.projectName, "Project Name", log);
-	await nAtATime(10, packages, pkg => checkNpm(pkg, log), {
-		name: "Checking for typed packages...",
-		flavor: pkg => pkg.typingsPackageName
-	});
+	if (includeNpmChecks) {
+		await nAtATime(10, packages, pkg => checkNpm(pkg, log), {
+			name: "Checking for typed packages...",
+			flavor: pkg => pkg.typingsPackageName
+		});
+	}
 	await writeLog("conflicts.md", logResult());
 }
 

packages/types-publisher/src/full.ts

@@ -25,7 +25,7 @@ export default async function full(client: NpmClient, dry: boolean, timeStamp: s
 	await clean();
 	await getDefinitelyTyped(options);
 	await parseDefinitions(options);
-	await checkParseResults();
+	await checkParseResults(/*includeNpmChecks*/ false);
 	await calculateVersions(/*forceUpdate*/ false, options);
 	await generatePackages(options);
 	await createSearchIndex(/*skipDownloads*/ false, /*full*/ false);

packages/types-publisher/src/tester/test.ts

@@ -16,6 +16,6 @@ if (!module.parent) {
 async function main(options: Options, nProcesses?: number): Promise<void> {
 	await clean();
 	await parseDefinitions(options);
-	await checkParseResults();
+	await checkParseResults(/*includeNpmChecks*/false);
 	await runTests(options, nProcesses);
 }