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

feat: any swift format #734

Merged
merged 10 commits into from
Dec 22, 2021
Merged

feat: any swift format #734

merged 10 commits into from
Dec 22, 2021

Conversation

gabischima
Copy link
Contributor

@gabischima gabischima commented Nov 16, 2021

Issue #, if available: could help #719

Description of changes:

  • ✨ add new format ios-swift/any.swift with:
    • objectType: type for the swift object - default is class
    • import: array or string to import modules - default is ["UIKit"]
    • accessControl: access level for the object - default is public
  • 🎨 update ios-swift/class and ios-swift/enum to import any.swift template
  • 🔥 remove class.swift.template and enum.swift.template
  • ✨ add helper setSwiftFileProperties to format file object for swift
  • ✅ add test for setSwiftFileProperties and snapshot for swift template
  • 📝 update docs comment for ios-swift formats
  • 📝 add ios-swift/any.swift to examples / basic / config.json

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Copy link
Member

@dbanksdesign dbanksdesign left a comment

Choose a reason for hiding this comment

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

Overall this looks great! Just a few comments and we can get this merged in. If you are ok with it, when we merge this in we would also like to give credit to @antoniogamizbadger because they started a similar PR as well.

__tests__/formats/swiftFile.test.js Outdated Show resolved Hide resolved
lib/common/formats.js Show resolved Hide resolved
lib/common/formatHelpers/setSwiftFileProperties.js Outdated Show resolved Hide resolved
lib/common/formatHelpers/setSwiftFileProperties.js Outdated Show resolved Hide resolved
@chazzmoney
Copy link
Collaborator

@gabischima Would love to get this one merged next week. Any chance you could make the requested changes?

Thanks!

@gabischima
Copy link
Contributor Author

@gabischima Would love to get this one merged next week. Any chance you could make the requested changes?

yes ! (: i will go into that

@gabischima
Copy link
Contributor Author

If you are ok with it, when we merge this in we would also like to give credit to @antoniogamizbadger because they started a similar PR as well.

yes of course !

@dbanksdesign dbanksdesign merged commit 9859a8d into amzn:main Dec 22, 2021
@gabischima gabischima deleted the feature/any-swift branch January 3, 2022 20:48
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.

4 participants