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

Re-merge swift lint #18

Closed
wants to merge 6 commits into from
Closed

Re-merge swift lint #18

wants to merge 6 commits into from

Conversation

anglinb
Copy link
Collaborator

@anglinb anglinb commented Mar 7, 2022

No description provided.

yusuftor and others added 5 commits March 1, 2022 18:25
- Lots of style changes to keep in line with swiftlint. It now runs the swiftlint file in the root of the folder on build.
- Split some types into dedicated files. This is "in progress".
@anglinb
Copy link
Collaborator Author

anglinb commented Mar 7, 2022

👋 Hey @yusuftor! Great job so far!

We merged in your changes but ran into errors when trying to run the sample app. We looked into it and realized that you had changed the names of the two variables in Product. Since this is coming from the API this make decoding responses fail. We reverted the changes on master and re-opened this PR.

https://github.com/superwall-me/paywall-ios/blob/dcee82a66adbb0ab059910df0f17fca730e97342/Sources/Paywall/Types/PaywallTypes/Product.swift#L16-L19

Would you mind double checking your changes work correctly in the sample app? We'll do a testing push once they're ready but some of the simple cases will demonstrate this issue. Don't worry about the super nuanced cases we can run those together, just a few smoke test ones.

@yusuftor
Copy link
Collaborator

yusuftor commented Mar 8, 2022

Thanks! Yes this hasn’t been tested with the test app yet so shouldn’t be merged with master just yet. Good spot on that one and it’s simple to fix. Will do a run through to see if there are any other codables I’ve accidentally changed too.

Added CodingKeys to Product
@jakemor jakemor closed this Mar 12, 2022
@yusuftor yusuftor deleted the feature/swiftlint-integration branch November 15, 2022 15:18
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.

3 participants