-
-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: replacing source code with typescript #13
base: master
Are you sure you want to change the base?
Conversation
Привет! ПС: Еще в пр изменились все файлы, возможно из-за переносов строк. Это в любом случае лучше убрать. |
Очень странно почему так вышло. Поправлю. Для чего нужен stryker? В master ветке он у меня не заработал |
c20c565
to
b525a40
Compare
Можно и так, да.
Stryker хорошая тулза, позволяет прогонять мутационные тесты. Но после какого-го релиза он сломался и пока действительно не работает. |
В диффе еще много изменений, которые не связаны с переходом на тс.
Это очень усложняет ревью пулл реквеста. Нужно оставить только то, что связано непосредственно с тс. А остальное лучше делать отдельными пр. |
b525a40
to
1e88880
Compare
Убрал лишнее и постарался не добавлять лишних пробелов. |
"lint": "eslint src test", | ||
"test": "mocha test/setup.js test/specs/*.spec.js", | ||
"watch": "rollup -c -w", | ||
"build": "rimraf dist/* && npx cross-env NODE_ENV=production rollup -c", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
rimraf нужно добавить в зависимости
package.json
Outdated
@@ -36,21 +36,28 @@ | |||
"node": ">=8" | |||
}, | |||
"lint-staged": { | |||
"{src,test}/**/*.js": "eslint" | |||
"{src,test}/**/*.(js,cjs,ts)": "eslint" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
А для чего оставляем js?
src/br.ts
Outdated
* - space in tts | ||
*/ | ||
import {textTts} from './text-tts'; | ||
import {Response} from "./reply"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Кавычки везде лучше сделать одинарные. Можно в eslint правило добавить, если еще нет.
Несколько комментов по коду написал.
Тогда давай это сделаем отдельным подготовительным ПР-ом. |
No description provided.