-
Notifications
You must be signed in to change notification settings - Fork 113
apply swift-format #380
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
apply swift-format #380
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
formatting still fails
Will use the NIO formatting script as suggested. |
@sebsto yes. this is because of an "old" toolchain. I applied the necessary changes manually. On linux it already works as intended (see CI). |
@fabianfett swift format is fixed ! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Your local swift format seems to have an issue with package. Those breaks aren't needed. In #379 CI passed without those weird breaks.
…bda-runtime into sebsto/swift-format
Fixed - not sure where it came from, maybe my first pass of swift-format without using swift-nio's script |
Apply swift-format to
Sources
andPlugins
directories