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.

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

Use SwiftLint to enforce Swift style and conventions #2300

Merged
merged 6 commits into from
Feb 27, 2019
Merged

Conversation

SBiOSoftWhare
Copy link
Contributor

No description provided.

@@ -71,7 +71,7 @@ PODS:
- GZIP (~> 1.2.2)
- libbase58 (~> 0.1.4)
- OLMKit (~> 3.0.0)
- Realm (~> 3.13.1)
- Realm (~> 3.11.1)
Copy link
Member

Choose a reason for hiding this comment

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

Why did realm get downgraded?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

To make pod install working with MatrixKit 0.9.5

Copy link
Member

Choose a reason for hiding this comment

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

riot-ios/develop should use pods of kit/develop & sdk/develop

Copy link
Contributor Author

Choose a reason for hiding this comment

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

If you uncomment $matrixKitVersion = 'develop' and perform a pod install it will update the podfile.lock correctly and bump Realm to 3.13.1. The podfile.lock here was generated by using default kit version $matrixKitVersion = '0.9.5' in podfile.

Copy link
Member

Choose a reason for hiding this comment

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

If you uncomment $matrixKitVersion = 'develop' and perform a pod install it will update the podfile.lock correctly and bump Realm to 3.13.1.

yes. This is what I meant to avoid to do a stupid rm Podfile.lock the day of the release.

- Riot/Generated/

line_length:
warning: 250
Copy link
Member

Choose a reason for hiding this comment

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

250 seems huge. Error on 160 seems more reasonable.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes it seems huge but several delegate methods and SwiftGen constants are near 200 characters

Copy link
Member

Choose a reason for hiding this comment

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

several delegate methods
This is another reason I hate them but well :/

@SBiOSoftWhare SBiOSoftWhare merged commit 94f69d9 into develop Feb 27, 2019
@SBiOSoftWhare SBiOSoftWhare deleted the swift_lint branch February 27, 2019 10:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants