Protractor: Пакет выпуска NPM должен заменить globals.ts на globals.d.ts

Созданный на 18 авг. 2016 · 3Комментарии · Источник: angular/protractor

Сообщение об ошибке

Версия узла: 6.3.1
Версия транспортира: 4.0.3
Версия машинописного текста: 2.0.0
Обзор

Это характерно для проектов машинописного текста с исходными файлами, импортирующими protractor/globals . В частности, при импорте protractor/globals , поскольку выпуск NPM включает файл globals.ts вместо соответствующего файла определения ( globals.d.ts ), компилятор машинописного текста вычислит общий исходный корень проекта машинописного текста. к общему пути между зависимостью Protractor global.ts (вероятно, node_modules/protractor ) и везде, где содержатся источники потребляющего проекта (например, src/test/**/*.ts ). Это заставляет компилятор выводить исходные файлы со всеми родительскими каталогами.

Обратите внимание, что это чисто проблема с тем, где компилятор машинописного текста выводит скомпилированные файлы; компиляция происходит нормально, и транспортир потребляется (как здорово), как и ожидалось.

Действия по воспроизведению

Чтобы лучше продемонстрировать сценарий, который я плохо описываю, см. Этот пример проекта .

Предложенное решение

Я подозреваю, что решение этой проблемы здесь было бы комбинацией двух вещей:

обновить текущий процесс компиляции gulp, чтобы обеспечить создание файла определения при компиляции файла globals.ts ; а также
обновить процесс выпуска Protractor NPM (вручную или автоматически), чтобы удалить файл globals.ts из пакета выпуска NPM, оставив только скомпилированный файл javascript и соответствующий файл определения.

Если это нормально, я с удовольствием отправлю запрос на перенос для обработки первой части решения. Если процесс выпуска для NPM по-прежнему следует документу о globals.ts , хотя это может быть лучше исправлено кем-то, кто действительно выполняет выпуски.

Хотя я обдумал это и считаю, что компилятор Typescript ведет себя так, как ожидалось, дайте мне знать, если вы не согласны, так как я все еще не очень хорошо разбираюсь в Typescript.

(править: формат обновления для отражения шаблона ошибки)

Источник

tmeneau

Самый полезный комментарий

@cnishina рада, что этот пример помог! Спасибо за быстрый ответ, это выглядит великолепно, за исключением одной вещи, в которой я могу быть сбит с толку: пока ваше изменение удаляет файл global.ts из пакета NPM, не будет соответствующего объявления globals.d.ts сделать protractor/globals import невидимым для компилятора машинописного текста для использования проектов?

Итак, я предполагаю, что первая часть моего предложения заключалась не в том, чтобы удалить скомпилированный globals.js в пользу объявления globals.d.ts , а в том, чтобы иметь оба, чтобы пакет NPM содержал:

globals.js
globals.d.ts

только без исходного файла globals.ts .

Я считаю, что этого можно достичь с помощью tsc с флагом --declaration при компиляции файла global.ts . Например, изменение задачи tsc: globals gulp, как tsc и tsc:w которые, возможно, также необходимо обновить при условии, что это не приведет к неблагоприятному изменению неглобальных компиляций:

gulp.task('tsc:globals', function(done) {
  runSpawn(done, 'node', ['node_modules/typescript/bin/tsc', '--declaration', 'globals.ts'],
    'ignore');
});

Сообщите мне, если я что-то упустил, и еще раз спасибо за быстрое исправление! Между прочим, использование транспортира с Typescript - это серьезная проблема!

tmeneau 18 авг. 2016

🎉1 👍1

Все 3 Комментарий

@tmeneau Спасибо за этот вопрос: мне очень понравился пример github ... 👍 👍 👍

Так что это побочный эффект при использовании outDir. Например, если для outDir задано значение output , в дополнение к перенесенным файлам, будет также перенесен output/node_modules/protractor/globals.js . Первоначально я включил файл globals.ts, поскольку (в то время) после некоторых экспериментов вы не могли импортировать типы файлов без файла TypeScript.

Чтобы просмотреть предложения:

Не работает, поскольку globals.ts не может быть просто файлом объявления * d.ts, поскольку мы назначаем переменные из глобального пространства имен.
После экспериментов с exampleTypescript похоже, это сработало. Я обновляю .npmignore, чтобы не публиковать этот файл, и обновляю Protractor до TypeScript 2. См. Https://github.com/angular/protractor/pull/3485

cnishina 18 авг. 2016

globals.js
globals.d.ts

только без исходного файла globals.ts .

gulp.task('tsc:globals', function(done) {
  runSpawn(done, 'node', ['node_modules/typescript/bin/tsc', '--declaration', 'globals.ts'],
    'ignore');
});

tmeneau 18 авг. 2016

🎉1 👍1

Ага ... 👍 globals.d.ts все еще должен быть там, чтобы protractor/globals import работал. Я обновил PR. Спасибо!

cnishina 19 авг. 2016

👍1

Была ли эта страница полезной?

0 / 5 - 0 рейтинги