Skip to content
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

Добавить .editorconfig #113

Merged
merged 2 commits into from
Nov 18, 2019
Merged

Добавить .editorconfig #113

merged 2 commits into from
Nov 18, 2019

Conversation

rafaelldi
Copy link
Contributor

В ишью #19 говориться о StyleCop, но сейчас editorconfig вроде более популярный инструмент.

Взял как шаблон файл из этого репозитория и поправил его в соответствии с дефолтными настройками студии (раз, два). В основном там все warning меняются на suggestion или silent.

@rafaelldi rafaelldi changed the title Issue #19 Добавил .editorconfig Issue 19 Добавил .editorconfig Nov 10, 2019
@kulakovt kulakovt changed the title Issue 19 Добавил .editorconfig Добавить .editorconfig Nov 10, 2019
@kulakovt
Copy link
Member

EditorConfig не заменяет StyleCop. Он его дополняет. И тоже конечно нужен.

Лучше взять за основу что-нибудь от MS, например от команды CoreFx.

EditorConfig - проверяет синтаксис
StyleCop (а нынче это DotNetAnalyzers) - проверяют семантику.

@Sergey-Buyanov
Copy link
Contributor

Sergey-Buyanov commented Nov 11, 2019

Добавлю,
В VS есть встроенные анализаторы, (ошибки с номерами IDExxxx), они берут правила из editorconfig.
Также есть дополнительные
StyleCop - работает на уровне исходного кода (в том числе, проверяет стиль кодирования форматирование и т.п.)
FxCop - делает анализ кода уже на уровне бинарного кода
Также можно добавить и другие Sonar, XUnit analyzerz, Resharper
Информация по Code Analyzers
В editorconfig также можно настраивать правила, для StyleCop, Resharper, etc.
Как-то настраивал правила здесь
Можно взять также правила по умолчанию, которые сгенерирует студия

@rafaelldi
Copy link
Contributor Author

Ок, спасибо, сразу не разобрался. Поменял .editorconfig на файл из репозитория corefx.
Попробовал локально добавить нугеты с FxCop и StyleCop.Analyzers, там получается очень много warning'ов.

@egorikas
Copy link
Contributor

Ну значит будем править warning

@rafaelldi
Copy link
Contributor Author

StyleCop добавлять в рамках этого PR или оставить его только для editorconfig?

@egorikas
Copy link
Contributor

@ilabutin мержим? как по мне, .editorconfig из репы .net core - это ок

@ilabutin
Copy link
Collaborator

StyleCop отдельно. Этот окей. Вмержу

@ilabutin ilabutin merged commit 17f0816 into DotNetRu:master Nov 18, 2019
@rafaelldi rafaelldi deleted the Issue-19-EditorConfig branch January 23, 2020 19:05
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.

5 participants