Tslint: Запрос функции: сводный отчет

Созданный на 28 февр. 2017  ·  20Комментарии  ·  Источник: palantir/tslint

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

  • __TSLint версия__: 4.50
  • __TypeScript version__:
  • __ Запуск TSLint через__: CLI

Было бы неплохо увидеть, как файлы линтируются, как это может делать eslint:
DEBUG=eslint:* eslint .

возможно вариант tslint --showfiles ?

Линтинг кода TypeScript

// code snippet

с конфигурацией tslint.json :


Фактическое поведение

Ожидаемое поведение

In Discussion Needs Proposal Enhancement 🌹 R.I.P. 🌹

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

Некоторые новости об этом? Хотелось бы хоть какой-нибудь простой отчет вроде @chrismbarr sample ...

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

Вы можете показать имя файла, используя средство подробного форматирования (https://palantir.github.io/tslint/formatters/verbose/)

ты уверен в этом. Verbose, похоже, показывает только файлы с ошибками. Я хочу показать все файлы с линзами, независимо от того, есть ли в них ошибки или нет

ах, извините, я неправильно понял запрос.

Я в порядке с флагом, но не уверен, что печать на консоль без форматирования - правильный подход.

Возможно, мы могли бы добавить еще одну функцию в IFormatter, и средства форматирования могли бы при желании реализовать их печать по-своему.

да, это звучит как хорошая идея

так что еще одна функция, кроме:
format(failures: RuleFailure[], fixes?: RuleFailure[]): string;

как
show(files: Files[]): string;

??

Я не особо жду здесь. Вы можете предоставить код или?

хм, а как насчет format(failures: RuleFailure[], fixes?: RuleFailure[], files?: string[]): string;

Я до сих пор не понимаю, зачем вам эта функция. Вы просто проверяете расширение своего глобуса?

Я использую это в автоматизации CI, чтобы убедиться, что все файлы ts были линтированы и что glob и все игнорируемые и т. Д. Работают

Учитывая, что у нас нет https: //github.com/palantir/tslint/issues/73 ... Я не думаю, что нам нужно реализовать это в самом TSLint. Можете ли вы проверить конфигурацию списка файлов tslint вне команды tslint в CI?

Я использовал:
tslint src/**/*.ts

но, видимо, никаких файлов не было
Я должен был использовать:
tslint 'src/**/*.ts'

так что вот список линтированных файлов был бы действительно хорош :)

Что, если бы у него было только окончательное резюме, в котором говорилось что-то вроде:

TSLint завершил линтинг 25 файлов, в 3 есть ошибки.

могло быть хорошо. список файлов лучше бы ИМХО

Некоторые новости об этом? Хотелось бы хоть какой-нибудь простой отчет вроде @chrismbarr sample ...

Отчет был бы отличным. Он работал на моем компьютере с Windows, но все остальные разработчики здесь используют Mac / Linux, включая сервер сборки, поэтому ошибки tslint остались незамеченными!

Добавление одинарных кавычек (как сказал @gemal )

Похоже, эта проблема превратилась в версию # 3941: Запрос функции: сводный отчет. Измените название здесь, чтобы сохранить более глубокое обсуждение.

Хотелось бы увидеть более подробное предложение именно того, что мы здесь добавляем.

Изменить: здесь связаны как дубликаты ...

  • Регистрация продолжительности отдельных правил
  • Количество сообщений о tslint:disable вхождениях
  • Регистрация всех известных правил TSLint
  • Регистрация всех _enabled_ правил TSLint

Я хочу показать все файлы с линзами, независимо от того, есть ли в них ошибки или нет

@gemal, чтобы добиться этого, вы можете написать собственное правило tslint, чтобы консоль регистрировала каждый файл, который она обрабатывает.

У меня было аналогичное требование, поэтому я написал это правило:

https://www.npmjs.com/package/tslint-log

Возможно, имеет отношение к обсуждению: https://github.com/karfau/tslint-report

@JoshuaKGoldberg спасибо за внимание к этой проблеме.
Насколько я понимаю из описания проблемы / разговора, первоначальное намерение касалось количества файлов, которые были добавлены. Это не то, что предоставляет tslint-report .
Вместо этого он больше связан со следующими темами:

  • Регистрация всех известных правил TSLint
  • Регистрация всех включенных правил TSLint

Это вы добавили в эту ветку. Но в этом сообщении мне непонятно, как, по вашему мнению, выложенные вопросы просят об этом.

Для полноты: этот набор правил использует tslint-report и добавляет его отчеты в свой репозиторий:

💀 _ Пора! _ 💀

TSLint устарел и больше не принимает запросы на вытягивание для основных новых изменений или функций. См. № 4534. 😱

Если вы хотите, чтобы это изменение было реализовано, у вас есть два варианта:

  • Рекомендуется: проверьте, доступно ли это в ESLint + typescript-eslint
  • _Не рекомендуется: форк TSLint локально _ 🤷‍♂️

👋 Было приятно работать с вами!

_Если вы считаете, что это сообщение было размещено здесь по ошибке, прокомментируйте, чтобы мы могли повторно открыть проблему! _

🤖 Бип-буп! 👉 TSLint устарел 👈 _ (# 4534) _, и вам следует переключиться на typescript-eslint ! 🤖

🔒 Эта проблема заблокирована, чтобы предотвратить дальнейшее ненужное обсуждение. Спасибо! 👋

Была ли эта страница полезной?
0 / 5 - 0 рейтинги