Skip to content

Logger: NSLock guaranteed#86

Open
kieran-osgood-shopify wants to merge 1 commit into
kieran-osgood/swift6/configure-only-checkoutkit-statefrom
kieran-osgood/swift6/synchronous-logger-state
Open

Logger: NSLock guaranteed#86
kieran-osgood-shopify wants to merge 1 commit into
kieran-osgood/swift6/configure-only-checkoutkit-statefrom
kieran-osgood/swift6/synchronous-logger-state

Conversation

@kieran-osgood-shopify

@kieran-osgood-shopify kieran-osgood-shopify commented May 13, 2026

Copy link
Copy Markdown
Contributor

What changes are you making?

Not much to say about this PR other than it reuses the LockedValue generic class from #85 to make concurrent access to the shared logger concurrency safe

How to test


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 (platforms/swift/README.md and/or platforms/android/README.md)

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

Tip

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

kieran-osgood-shopify commented May 13, 2026

Copy link
Copy Markdown
Contributor Author

@github-actions

github-actions Bot commented Jun 10, 2026

Copy link
Copy Markdown

React Native — Coverage Report

Lines Statements Branches Functions
Coverage: 92%
91.66% (319/348) 87.86% (181/206) 100% (82/82)

@kieran-osgood-shopify kieran-osgood-shopify force-pushed the kieran-osgood/swift6/synchronous-logger-state branch from 1b2c323 to a9cef3b Compare June 11, 2026 08:54
@kieran-osgood-shopify kieran-osgood-shopify force-pushed the kieran-osgood/swift6/configure-only-checkoutkit-state branch from 0763c3b to 0955280 Compare June 11, 2026 09:33
@kieran-osgood-shopify kieran-osgood-shopify force-pushed the kieran-osgood/swift6/synchronous-logger-state branch from a9cef3b to 0690561 Compare June 11, 2026 09:33
@kieran-osgood-shopify kieran-osgood-shopify force-pushed the kieran-osgood/swift6/configure-only-checkoutkit-state branch from 0955280 to c7e8091 Compare June 11, 2026 10:01
@kieran-osgood-shopify kieran-osgood-shopify force-pushed the kieran-osgood/swift6/synchronous-logger-state branch from 0690561 to 29a0e02 Compare June 11, 2026 10:01
@kieran-osgood-shopify kieran-osgood-shopify force-pushed the kieran-osgood/swift6/configure-only-checkoutkit-state branch from c7e8091 to 945d4b6 Compare June 11, 2026 10:23
@kieran-osgood-shopify kieran-osgood-shopify force-pushed the kieran-osgood/swift6/synchronous-logger-state branch from 29a0e02 to 673bb39 Compare June 11, 2026 10:23
@kieran-osgood-shopify kieran-osgood-shopify force-pushed the kieran-osgood/swift6/configure-only-checkoutkit-state branch from 945d4b6 to fe87622 Compare June 11, 2026 10:32
@kieran-osgood-shopify kieran-osgood-shopify force-pushed the kieran-osgood/swift6/synchronous-logger-state branch from 673bb39 to 29722dd Compare June 11, 2026 10:32
@kieran-osgood-shopify kieran-osgood-shopify changed the base branch from kieran-osgood/swift6/configure-only-checkoutkit-state to graphite-base/86 June 11, 2026 12:06
@kieran-osgood-shopify kieran-osgood-shopify force-pushed the kieran-osgood/swift6/synchronous-logger-state branch from 29722dd to 7c8db7f Compare June 11, 2026 13:08
@kieran-osgood-shopify kieran-osgood-shopify changed the base branch from graphite-base/86 to kieran-osgood/swift6/configure-only-checkoutkit-state June 11, 2026 13:08
@kieran-osgood-shopify kieran-osgood-shopify marked this pull request as ready for review June 11, 2026 14:31
@kieran-osgood-shopify kieran-osgood-shopify requested a review from a team as a code owner June 11, 2026 14:31
@kieran-osgood-shopify kieran-osgood-shopify force-pushed the kieran-osgood/swift6/configure-only-checkoutkit-state branch from 63e7604 to b28434d Compare June 12, 2026 10:00
@kieran-osgood-shopify kieran-osgood-shopify force-pushed the kieran-osgood/swift6/synchronous-logger-state branch 2 times, most recently from f8b8244 to db98f08 Compare June 12, 2026 10:58
@kieran-osgood-shopify kieran-osgood-shopify force-pushed the kieran-osgood/swift6/configure-only-checkoutkit-state branch from b28434d to 1d9c9a8 Compare June 12, 2026 10:58
@kieran-osgood-shopify kieran-osgood-shopify changed the title Make logger shared state concurrency-safe CheckoutKit UI: @MainActor isolatedMake logger shared state concurrency-safe Jun 12, 2026
@kieran-osgood-shopify kieran-osgood-shopify changed the title CheckoutKit UI: @MainActor isolatedMake logger shared state concurrency-safe CheckoutKit Logger: NSLock guaranteed Jun 12, 2026
@kieran-osgood-shopify kieran-osgood-shopify force-pushed the kieran-osgood/swift6/configure-only-checkoutkit-state branch from 1d9c9a8 to bf73472 Compare June 12, 2026 12:54
@kieran-osgood-shopify kieran-osgood-shopify force-pushed the kieran-osgood/swift6/synchronous-logger-state branch from db98f08 to 157151b Compare June 12, 2026 12:54
@kieran-osgood-shopify kieran-osgood-shopify changed the title CheckoutKit Logger: NSLock guaranteed Logger: NSLock guaranteed Jun 12, 2026
@kieran-osgood-shopify kieran-osgood-shopify force-pushed the kieran-osgood/swift6/synchronous-logger-state branch from 157151b to 488079a Compare June 12, 2026 14:11
@kieran-osgood-shopify kieran-osgood-shopify force-pushed the kieran-osgood/swift6/configure-only-checkoutkit-state branch from bf73472 to 1b9f6d6 Compare June 12, 2026 14:11
@kieran-osgood-shopify kieran-osgood-shopify changed the base branch from kieran-osgood/swift6/configure-only-checkoutkit-state to graphite-base/86 June 12, 2026 15:21
@kieran-osgood-shopify kieran-osgood-shopify force-pushed the kieran-osgood/swift6/synchronous-logger-state branch from 488079a to de784be Compare June 12, 2026 15:22
@kieran-osgood-shopify kieran-osgood-shopify changed the base branch from graphite-base/86 to kieran-osgood/swift6/configure-only-checkoutkit-state June 12, 2026 15:22
@kieran-osgood-shopify kieran-osgood-shopify force-pushed the kieran-osgood/swift6/synchronous-logger-state branch from de784be to 329a1a3 Compare June 12, 2026 15:31
@kieran-osgood-shopify kieran-osgood-shopify force-pushed the kieran-osgood/swift6/configure-only-checkoutkit-state branch from 1f63d02 to 0d6d8de Compare June 12, 2026 15:40
@kieran-osgood-shopify kieran-osgood-shopify force-pushed the kieran-osgood/swift6/synchronous-logger-state branch from 329a1a3 to 88ac844 Compare June 12, 2026 15:40
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.

1 participant