Material-components-ios: [Scripts] Linting and Formatting

Created on 11 Nov 2016  路  6Comments  路  Source: material-components/material-components-ios

There's no easy linting and formatting solution anymore in our process. clang-format should really be run.

This should be scripted.

Needs actionability review Scripts Bug Feature request Process
Source
picture willlarche

All 6 comments

@willlarche This is less important than having clang-format automatically format the core library, but how do you feel about including SwiftLint where relevant?

samsymons picture samsymons on 4 Jan 2017

I believe that's a great idea. SwiftLint has a lot going for it.

willlarche picture willlarche on 4 Jan 2017

Nice, I will try to open up a PR with some changes there today. We've been using SwiftLint a lot at PayPal and Venmo and it's been super useful!

samsymons picture samsymons on 4 Jan 2017
🎉1

https://github.com/seanoshea/BostonFreedomTrail/issues/126 for where I've introduced SwiftLint in a different project. Definitely useful!

seanoshea picture seanoshea on 4 Jan 2017
👍1

I have done a basic integration of SwiftLint in #1129.

samsymons picture samsymons on 5 Jan 2017
👍1

Closing as ignored. 馃槥
Please open a new bug if this is still an issue. 馃槃

ianegordon picture ianegordon on 24 Aug 2018
Was this page helpful?
0 / 5 - 0 ratings

Related issues

brianjmoore picture brianjmoore  路  5Comments
zishanj picture zishanj  路  4Comments
steviemo picture steviemo  路  7Comments
material-mirror picture material-mirror  路  10Comments
temkich picture temkich  路  8Comments