Skip to content

Syntax validation#457

Open
ITStan wants to merge 5 commits intooknosoft:developfrom
ITStan:syntax-validation
Open

Syntax validation#457
ITStan wants to merge 5 commits intooknosoft:developfrom
ITStan:syntax-validation

Conversation

@ITStan
Copy link
Copy Markdown

@ITStan ITStan commented Aug 26, 2020

правки по линтеру в шторме + явное указание опциональных параметров.

@unpete
Copy link
Copy Markdown
Member

unpete commented Aug 28, 2020

@ITStan, благодарю за вклад.
Просьба разбить PR на отдельные кусочки. Например, по коммитам.
У нас нет автотестов, а изменения сразу 15 файлов ко мне в голову не помещаются. В файле packages/metadata-core/src/mngrs.js отличается количество строк. Вероятно, изменен не только синтаксис.
Так же, предлагаю пока воздержаться от использования в ядре конструкции ?. - есть клиенты со старыми версиями node.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants