-
Notifications
You must be signed in to change notification settings - Fork 121
WIP: Йода-синтаксис. Fix #291 #297
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
Conversation
… аст и как по нему бегать и проверять.
|
У нас же теперь естьlVal! Или как его там, и диагностику можно сделать нормально, без регулярки! |
|
lValue - это про ассайнмент. а здесь экспрешен внутри условия, не? |
Угу, я тупой =( |
| * Annotation Processors | ||
| * Enable annotation processing - checked | ||
|
|
||
| ## Добавление диагностики |
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.
Это лишнее, есть отдельный раздел с документацией
| Сравнивая значения и переменные помни: переменные слева быть должны (С) Yoda | ||
|
|
||
| Плохо: | ||
| ```BSL |
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.
надо исправить на bsl
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.
Описание стоит скорректировать по шаблону
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.
и файлы переименовать по шаблону
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.
оно начиналось еще в августе. тут многое уже надо дотянуть до шаблона :)
| scope = DiagnosticScope.BSL, | ||
| severity = DiagnosticSeverity.MINOR, | ||
| minutesToFix = 2 | ||
| ) |
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.
нужны тэги
| @@ -1,5 +1,8 @@ | |||
| <?xml version="1.0" encoding="UTF-8"?> | |||
| <project version="4"> | |||
| <component name="JavaScriptSettings"> | |||
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.
этот файл стоит убрать ... @nixel2007 ?
|
Я попробую дотянуть. |
|
Не хватило прав на форспуш. |
Даигностика не проходит тест. Не могу понять, толи диапазон не правильный толи ошибка в коде. Как делать отладку не понял.
В комит попали файлы настроек, не понял почему.
.gitignor немного упорядочил.