Repository navigation
Приложение для терминала - #2
Dmitiry1921 merged 3 commits into
Conversation
Dmitiry1921
left a comment
Dmitiry1921
left a comment
There was a problem hiding this comment.
Оставил коменты в ПР
| @@ -0,0 +1,4 @@ | |||
| export interface Command { | |||
| getName(): string; | |||
| execute(...parameters: string[]): void; | |||
There was a problem hiding this comment.
Вот тут можно сделать опциональный параметр parameters так как не везде его принимать нужно, например help и version у тебя не принимают никаких параметров
There was a problem hiding this comment.
Дк вроде rest параметр и так не обязательный, ну тоесть он создаст пустой массив, если в аругменты при вызхове ничего не передать, я вроде из учебного проекта это взял)
| extends: | ||
| - "plugin:@typescript-eslint/recommended" | ||
| - "htmlacademy/node" | ||
| rules: |
There was a problem hiding this comment.
А эти правила зачем добавлял ? В требованиях задания вроде бы об этом ни слова )
There was a problem hiding this comment.
Мне режет глаз, когда скобки прилипают например {Obj}, просто добавил для своего удобства - вроде не наружшает каких-то правил?
There was a problem hiding this comment.
Нет правил не нарушает, просто удивился, что ты сразу в линтинг полез )
There was a problem hiding this comment.
А зачем тебе Makefile в NodeJS проекте ?
There was a problem hiding this comment.
Ну мне просто удобней через make команды писать - не принципиально - могу удалить)
6a6e4de to
a17d7b2
Compare
🎓 Приложение для терминала