Skip to content

Add documentation for .editorconfig properties#3374

Open
FantasyTeddy wants to merge 1 commit intoDotNetAnalyzers:masterfrom
FantasyTeddy:master
Open

Add documentation for .editorconfig properties#3374
FantasyTeddy wants to merge 1 commit intoDotNetAnalyzers:masterfrom
FantasyTeddy:master

Conversation

@FantasyTeddy
Copy link
Copy Markdown

I added some documentation for the .editorconfig properties introduced with #3285.

@codecov
Copy link
Copy Markdown

codecov bot commented Jul 28, 2021

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.43%. Comparing base (cedfcf9) to head (140f2b6).
⚠️ Report is 343 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3374      +/-   ##
==========================================
+ Coverage   94.34%   97.43%   +3.08%     
==========================================
  Files         920      920              
  Lines      109565   109592      +27     
  Branches     3306     3306              
==========================================
+ Hits       103373   106780    +3407     
+ Misses       5172     1838    -3334     
+ Partials     1020      974      -46     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@JobaDiniz
Copy link
Copy Markdown

Basically, all the json settings are supported in editorconfig file, right? You just need to "flat" the json path to create the key to be used in the editor config, is that right?

@FantasyTeddy
Copy link
Copy Markdown
Author

I think that's a good general summary, yes.
However, there are a few exceptions which cannot be set in the .editorconfig or use a different property name (e.g. from the .NET/C# formatting rules)

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