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.

Sign up for GitHub

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

Jump to bottom

Apply SwiftFormat for all files #2502

Merged
merged 48 commits into from
Feb 13, 2024
Merged

Apply SwiftFormat for all files #2502

merged 48 commits into from
Feb 13, 2024

Conversation

sosnovsky
Copy link
Collaborator

@sosnovsky sosnovsky commented Jan 19, 2024 •
edited
Loading

This PR applies SwiftFormat fixes to all files for consistent formatting and fixes SwiftLint warnings

Tests (delete all except exactly one):

  • Does not need tests (refactor only, docs or internal changes)

To be filled by reviewers

I have reviewed that this PR... (tick whichever items you personally focused on during this review):

  • addresses the issue it closes (if any)
  • code is readable and understandable
  • is accompanied with tests, or tests are not needed
  • is free of vulnerabilities
  • is documented clearly and usefully, or doesn't need documentation

@sosnovsky sosnovsky requested a review from DenBond7 February 12, 2024 19:48
@sosnovsky sosnovsky marked this pull request as ready for review February 12, 2024 19:48
@sosnovsky sosnovsky requested a review from tomholub as a code owner February 12, 2024 19:48
@sosnovsky
Copy link
Collaborator Author

@DenBond7 there is no functionality changes in this PR, just linter warning fixes and formatting updated with SwiftFormat

@sosnovsky sosnovsky enabled auto-merge (squash) February 12, 2024 19:50
@DenBond7
Copy link
Collaborator

@DenBond7 there is no functionality changes in this PR, just linter warning fixes and formatting updated with SwiftFormat

I'm reviewing right now

DenBond7
DenBond7 approved these changes Feb 13, 2024
View reviewed changes
Copy link
Collaborator

@DenBond7 DenBond7 left a comment

Choose a reason for hiding this comment

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

👍

@sosnovsky sosnovsky merged commit c3129cf into master Feb 13, 2024
11 checks passed
@sosnovsky sosnovsky deleted the 000-swiftformat branch February 13, 2024 07:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@DenBond7 DenBond7 DenBond7 approved these changes

@tomholub tomholub Awaiting requested review from tomholub

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sosnovsky @DenBond7