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

Swift Language Update #660

Merged
merged 3 commits into from
Feb 28, 2023
Merged

Conversation

maxxfrazer
Copy link
Contributor

  • Removed a couple of spaces that don't fit swift standard convention.
  • Removed one level of nested if, replaced with a continue.

[[String : Any]] -> [[String: Any]]` can be found here
Screenshot 2023-01-09 at 12 47 43

@maxxfrazer
Copy link
Contributor Author

Hey @umeshp7, are you able to take a look at this one?

@dhwaneetbhatt
Copy link
Contributor

Hey @maxxfrazer, we've made some changes in Swift codegen. Can you use the latest package once and see if we still require this PR? If yes, could you please resolve the conflicts in the file?

@maxxfrazer
Copy link
Contributor Author

Looks like most of this is still relevant IMO. Will fix conflicts and make sure it all looks good; will update you after.

@dhwaneetbhatt
Copy link
Contributor

@SahilChoudhary22 please review these changes once as you recently worked on Swift changes.

@maxxfrazer
Copy link
Contributor Author

@dhwaneetbhatt @SahilChoudhary22 appreciate it!

@dhwaneetbhatt
Copy link
Contributor

I have verified the SwiftGen works correctly as expected in a live code editor. Merging this PR. Thank you for contributing @maxxfrazer.

@dhwaneetbhatt dhwaneetbhatt merged commit 61426cf into postmanlabs:develop Feb 28, 2023
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.

None yet

2 participants