atls-lab / config Goto Github PK

Описание бага

@atlantis-lab/config не находит prettier config тк он описан в ESM формате, а вызывается через commonjs

Воспроизведение

import config from '@atlantis-lab/config/prettier.config'
console.log(config)

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

конфиг импортируется

Материалы

atls/tools#119 - задача родитель

Описание бага

Объекты конфигов экспортируется в неверном формате

Воспроизведение

Шаги для воспроизведения бага:

1. Запускаем линтер используя данный конфиг
2. Видим ошибку

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

Ошибок быть не должно

Материалы

Пример запуска jest используя конфиг из репы:

Unknown option "default" with value 
{"collectCoverageFrom": ["**/*.{js,jsx,ts,tsx}", "!**/*.d.ts"], "coverageDirectory": "<rootDir>/coverage", "globals": {"ts-jest": {"tsConfig": {"allowJs": true}}}, "moduleNameMapper": {"^.+\\.(jpg|jpeg|gif|png|mp4|mkv|avi|webm|swf|wav|mid)$": "jest-static-stubs/$1"}, "modulePathIgnorePatterns": ["dist"], "preset": "ts-jest/presets/js-with-ts", "rootDir": "/media/darius/6E4CBE7A4CBE3D21/atlantis/atlantis/actl/actl", "snapshotSerializers": ["jest-emotion"], "testRegex": "/(__tests__|e2e)/.*\\.(test|spec)\\.(ts|tsx|js)$"} was found.
  This is probably a typing mistake. Fixing it will remove this message.

Данные окружения

• [email protected]
• Chrome Version 88.0.4324.150 (Official Build) (64-bit)

Описание бага

В задачах #83 и atls/tools#100 проблемой был поломанный функционал пакетов. Его ломал метод экспорта конфигов через export default, так что временно его пришлось заменить на cjs'ный module.exports

При экспорте создавался объект с high-level property, что некорректно. В случае с export default high-level propety - это default, в котором леэал сам конфиг. Такое свойство так же создается при обычном export после компиляции кода.

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

Конфиги экспортируются с использованием ESM синтаксиса и все работает

Материалы

См. referenced issue выше

Данные окружения

• [email protected]

Описание бага

@atlantis-lab/[email protected]

Обновление некоторых зависимостей поломало обратную совместимость плагинов линтера, из-за чего тот не выполняет 90% своей работы

Воспроизведение

Шаги для воспроизведения бага:

1. Запускаем линтер с текущим конфигом в какой-нибудь большой репе
2. Видим, что большая часть файлов не обработалась

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

Линтер должен работать на всех файлах, и все плагины должны отрабатывать корректно

Материалы

Пример ошибки:

Error: Comment "eslint-disable" was not printed. Please report this error!

Вариаций ошибок много, но все появляются из-за того, что те или иные плагины не работают

Данные окружения

• [email protected]

Описание бага

1. @atlantis-lab/prettier-config@latest
2. actl commit

При попытке сделать коммит у нас на проектах висят husky с хуками на precommit и commit-msg, если в стейдже нет файлов мы ловим ошибку после чего коммит прерывается.

Воспроизведение

Шаги для воспроизведения бага:

1. Создай репу
2. Насри файлами
3. создай блок вида https://github.com/Atlantis-Lab/serenity/blob/dev/package.json#L32
4. git add .
5. actl commit

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

линт-стейджед ничего не нашёл, мы успешно коммитнули файлы

Материалы

husky > pre-commit (node v14.14.0)
Could not parse lint-staged config.

        Error: ● Validation Error:

Invalid value for 'default'.

Should be a string, a function, or an array of strings and functions.

Configured value is: {'*.{js,ts,tsx,yml,yaml,json,graphql,md}': ['prettier --write --config /Users/torinasakura/workspace/atlantis/serenity/node_modules/@atlantis-lab/prettier-config/dist/index.js --ignore-path /Users/torinasakura/workspace/atlantis/serenity/node_modules/@atlantis-lab/prettier-config/.prettierignore', 'git add'], '*.{ts,tsx}': ['bash -c "tsc --noEmit"'], '*.{js,jsx,ts,tsx}': ['eslint --fix --config /Users/torinasakura/workspace/atlantis/serenity/node_modules/@atlantis-lab/eslint-config/dist/index.js', 'jest --config /Users/torinasakura/workspace/atlantis/serenity/node_modules/@atlantis-lab/jest-config/dist/index.js --bail --findRelatedTests', 'git add']}

Please refer to https://github.com/okonet/lint-staged#configuration for more information...

Please make sure you have created it correctly.
See https://github.com/okonet/lint-staged#configuration.

husky > pre-commit hook failed (add --no-verify to bypass)
/Users/torinasakura/workspace/atlantis/serenity/node_modules/commitizen/dist/cli/strategies/git-cz.js:85
        throw error;
        ^

Error: git exited with error code 1
    at ChildProcess.<anonymous> (/Users/torinasakura/workspace/atlantis/serenity/node_modules/commitizen/dist/git/commit.js:807:28)
    at ChildProcess.emit (events.js:315:20)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:277:12) {
  code: 1,
  signal: null
}

Данные окружения

• nodeJS: 14.14.0
• yarn: 1.22.10
• atlantis-lab/actl: latest

Описание бага

Конфиги экспортируются без ключевого слова

export default {}

из-за чего @atlantis-lab/actl-lint и @atlantis-lab/actl-test не находят конфиг

Какого рода задача?

Убрать ненужное свойство из конфига

Что и где будем менять?

extends: ['airbnb-typescript', 'prettier/react', 'plugin:prettier/recommended'],

На

extends: ['airbnb-typescript', 'plugin:prettier/recommended'],

Укажите референс

atls/tools#104

Описание бага

@atlantis-lab/[email protected]

Старая зависимость мешает работе линтера

Воспроизведение

Шаги для воспроизведения бага:

1. Запускаем линтер с текущим конфигом
2. Получаем ошибку

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

Зависимость должна быть свежей, линтер должен работать

Материалы

TypeError: path.match is not a function

Данные окружения

• [email protected]

Описание бага

@atlantis-lab/[email protected]

Линтер перестал работать

Воспроизведение

Шаги для воспроизведения бага:

1. Запускаем линтер используя данный конфиг
2. Видим ошибку

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

Ошибок быть не должно и линтер должен запуститься

Материалы

Error: Error while loading rule '@typescript-eslint/dot-notation': 
You have used a rule which requires parserServices to be generated. 
You must therefore provide a value for the "parserOptions.project" property for @typescript-eslint/parser.

Данные окружения

• [email protected]