-
Notifications
You must be signed in to change notification settings - Fork 42
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
Issue#1250: Corrected alignment of multiline variable and signal declarations for align_left = no #1345
base: master
Are you sure you want to change the base?
Conversation
…ign_left_no_align_paren_yes.
…r method _analyze_align_left_no_align_paren_yes.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #1345 +/- ##
=======================================
Coverage 94.38% 94.38%
=======================================
Files 1749 1749
Lines 31000 31032 +32
Branches 3462 3453 -9
=======================================
+ Hits 29258 29289 +31
Misses 1328 1328
- Partials 414 415 +1 ☔ View full report in Codecov by Sentry. |
Evening @JHertz5 , Could you resolve the conflicts with master? Thanks, --Jeremy |
@jeremiah-c-leary I've updated the branch and resolved the merge conflicts. |
Resolves #1250.
I found that the extra space is present for "align_left_no__align_paren_no" and "align_left_no__align_paren_yes", so I've made adjustments to both.