Merge pull request #6846 from Microsoft/noErrorOnEmptyDtsFile

do not error if typings for external library is empty .d.ts file
This commit is contained in:
Vladimir Matveev
2016-02-02 12:12:52 -08:00
parent 175493423e
commit 9fe10a6ac0
5 changed files with 38 additions and 1 deletions

View File

@@ -1148,7 +1148,7 @@ namespace ts {
if (importedFile && resolution.isExternalLibraryImport) {
// Since currently irrespective of allowJs, we only look for supportedTypeScript extension external module files,
// this check is ok. Otherwise this would be never true for javascript file
if (!isExternalModule(importedFile)) {
if (!isExternalModule(importedFile) && importedFile.statements.length) {
const start = getTokenPosOfNode(file.imports[i], file);
fileProcessingDiagnostics.add(createFileDiagnostic(file, start, file.imports[i].end - start, Diagnostics.Exported_external_package_typings_file_0_is_not_a_module_Please_contact_the_package_author_to_update_the_package_definition, importedFile.fileName));
}