Skip to content

Conversation

@FantasyTeddy
Copy link

@FantasyTeddy FantasyTeddy commented Jul 23, 2025

This relaxes SA1516 as suggested in #2441 (comment):

  • If two properties are both single line, they do not require a blank line in between.

Since this change makes StyleCop behave less strict, I'm not sure if this behavior should be configurable. Let me know if it should.

Closes #2441

@codecov
Copy link

codecov bot commented Jul 23, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.39%. Comparing base (f5843ae) to head (e3ba459).
⚠️ Report is 86 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3936      +/-   ##
==========================================
- Coverage   97.44%   97.39%   -0.05%     
==========================================
  Files         926      953      +27     
  Lines      110283   111930    +1647     
  Branches     3319     3367      +48     
==========================================
+ Hits       107467   109018    +1551     
- Misses       1847     1927      +80     
- Partials      969      985      +16     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@sharwell sharwell enabled auto-merge November 25, 2025 14:33
@sharwell sharwell merged commit 0d90d66 into DotNetAnalyzers:master Nov 25, 2025
21 checks passed
@FantasyTeddy FantasyTeddy deleted the relaxed-sa1516 branch November 25, 2025 20:51
@sharwell sharwell added this to the 1.2-beta.next milestone Nov 25, 2025
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.

SA1516: Relaxed version available?

2 participants