Skip to content

Add quicktype package dependency to dev.yml#24

Merged
markmur merged 1 commit into
mainfrom
package-deps
May 8, 2026
Merged

Add quicktype package dependency to dev.yml#24
markmur merged 1 commit into
mainfrom
package-deps

Conversation

@markmur
Copy link
Copy Markdown
Contributor

@markmur markmur commented May 6, 2026

What changes are you making?

Adds quicktype as a package dependency to the dev.yml. The Model.swift file will be added in a follow up.


Before you merge

Important

  • I've added tests to support my implementation
  • I have read and agree with the Contribution Guidelines
  • I have read and agree with the Code of Conduct
  • I've updated the relevant platform README (swift/README.md and/or android/README.md)

Releasing a new Swift version?
  • I have bumped the version in swift/ShopifyCheckoutKit.podspec
  • I have bumped the version in swift/Sources/ShopifyCheckoutKit/ShopifyCheckoutKit.swift
  • I have updated swift/CHANGELOG.md
  • I have updated the SwiftPM/CocoaPods version snippets in swift/README.md (major version only)
Releasing a new Android version?
  • I have bumped the versionName in android/lib/build.gradle
  • I have updated android/CHANGELOG.md
  • I have updated the Gradle/Maven version snippets in android/README.md

Tip

See the Contributing documentation for the full release process per platform.

Copy link
Copy Markdown
Contributor Author

markmur commented May 6, 2026

This stack of pull requests is managed by Graphite. Learn more about stacking.

@markmur markmur force-pushed the package-deps branch 3 times, most recently from 1e6f4d5 to e66536a Compare May 6, 2026 19:26
Base automatically changed from 05-05-add_ecp_support_to_android_checkout_kit to main May 6, 2026 20:28
@markmur markmur marked this pull request as ready for review May 6, 2026 20:47
@markmur markmur force-pushed the package-deps branch 5 times, most recently from 4b9b2ae to 70a617d Compare May 8, 2026 09:55
@markmur markmur enabled auto-merge (squash) May 8, 2026 10:07
Comment thread .swiftlint.yml

excluded:
- swift/Samples
- swift/Sources/ShopifyCheckoutKit/Models.swift
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I'm not adding the generated file in this PR because CI is having issues with license, linting, compilation etc. Will add it in a follow up

@markmur markmur merged commit bf51adb into main May 8, 2026
10 checks passed
@markmur markmur deleted the package-deps branch May 8, 2026 10:09
Comment thread dev.yml

up:
- packages:
- quicktype
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Wondering if pnpx would be a better approach - dev up wont work in CI, and I think we want to automate the releases?

Approving anyway as we probably wont set that up in this week

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yeah happy to revisit this next week

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

#gsd:50662 Rebase Checkout Kit on UCP

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants