Skip to content

Add new format rule for wrapping case bodies#2476

Merged
calda merged 1 commit intonicklockwood:developfrom
kimdv:kimdv/add-wrapped-swift-case
Mar 24, 2026
Merged

Add new format rule for wrapping case bodies#2476
calda merged 1 commit intonicklockwood:developfrom
kimdv:kimdv/add-wrapped-swift-case

Conversation

@kimdv
Copy link
Contributor

@kimdv kimdv commented Mar 23, 2026

Hi!

In out codebase we have a rule stating that all case bodies need to be on a newline.
So my toughts was to add this SwiftFormat to enforce this rule, so other could benefit from this too.

Looking forward to the feedback :D

@codecov
Copy link

codecov bot commented Mar 23, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.30%. Comparing base (afd7c1f) to head (55c125e).
⚠️ Report is 1 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2476      +/-   ##
===========================================
+ Coverage    95.28%   95.30%   +0.02%     
===========================================
  Files          166      167       +1     
  Lines        25304    25330      +26     
===========================================
+ Hits         24112    24142      +30     
+ Misses        1192     1188       -4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Collaborator

@calda calda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@calda calda merged commit 518a4c1 into nicklockwood:develop Mar 24, 2026
16 checks passed
@kimdv kimdv deleted the kimdv/add-wrapped-swift-case branch March 24, 2026 08:10
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