-
Notifications
You must be signed in to change notification settings - Fork 35
merge(smithy-swift): Swift-client-runtime branch into master branch #65
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
Conversation
…ges in source code
chore: add swiftLint and codecov tooling to swift ClientRuntime
* feat(codegen): adding swift protocol generation and tests * style(codegen): fixed ktlint issues * test: fix tests to separate out pieces of protocol * style(codegen): fix ktlint errors one last time i hope
… minor formatting
* feat: add package manifest file generator * feat: tools support to test code generated sdk builds * fix: move swiftVersion to swiftSettings * fix: dependecies available as pods and swift packages have empty package type
…/smithy-swift into swift-client-runtime
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, just refactor folder structure after removal of smithy-kotlin and it's a go!
], | ||
dependencies: [ | ||
.package(path: "~/Projects/Amplify/SwiftSDK/aws-crt-swift") | ||
.package(path: "~/Documents/aws-crt-swift") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
circleci will fail I believe cuz you cloned to this path unless you changed to where you are cloning to in circleci?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My bad, reverting
Description of changes:
Merge Swift-client-runtime branch into master branch
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.