-
Notifications
You must be signed in to change notification settings - Fork 10.4k
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
Remove swift-indent
tool
#76577
Remove swift-indent
tool
#76577
Conversation
This removes the implementation of the `swift-indent` tool, its associated documentation, and utilities. This tool was never completed and has much better alternatives with `swift-format` which is more flexible and actually maintained.
@swift-ci please test |
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.
Makes sense to me!
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.
LGTM.
Do we need a matching patch to the driver repo to update the options there?
Yeah, it would be good to update swift-driver as well. |
@swift-ci please test Linux platform |
This is the companion of swiftlang/swift#76872. I've reused the `SwiftIndentOption` constant, which was removed by swiftlang/swift#76577 (but Options.swift was never updated here to remove it).
This is the companion of swiftlang/swift#76872. I've reused the `SwiftIndentOption` constant, which was removed by swiftlang/swift#76577 (but Options.swift was never updated here to remove it).
This removes the implementation of the
swift-indent
tool, its associated documentation, and utilities. This tool was never completed and has much better alternatives withswift-format
which is more flexible and actually maintained.