-
Notifications
You must be signed in to change notification settings - Fork 11
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
Add SwiftLint support #191
Comments
We should also explore swift-format |
Note: Swift lint is being added at the SDK level in 200.5 as a plugin. Ref Swift 2836. cc @zkline101 @mhdostal |
You may want to change this because once your Examples project is updated to the Xcode 15 recommended changes (it may already be upgraded), it will break that build phase. This is because Xcode will automatically change |
Note that we ended up adding SwiftLint as a custom package rather than a plugin. See Swift 5164. |
In today's discussion on using SwiftLint build tool plugin, there are some concerns about adding the plugin to the toolkit Swift Package
The benefits of SwiftLint build tool plugin as of today (200.4)
|
Esri fork of SwiftLint -> https://github.com/Esri/SwiftLint
Update: This can be consumed as a package plugin to lint package sources. The Toolkit example projects can add it as a package dependency $PROJECT -> Package Dependencies and then add a build tool plugin step under $TARGET -> Run Build Tool Plug-ins Once this work is done we should carry out #698 Edit: Possible new issue with the SwiftLint SPM plug-in that should be resolved first @ Swift 5296 |
The v100.x Toolkit had a SwiftLint build phase. We should add SwiftLint to the Toolkit, Example,
and AuthenticatorExample projects(the Authenticator Example is essentially static and unchanging, so we're not going to lint that). The Samples team has already done this for the Swift Samples, so follow that pattern.The text was updated successfully, but these errors were encountered: