Skip to content

Move React Native dev commands under root dev.yml#64

Open
kyle-schellen wants to merge 1 commit into
05-08-move_swift_dev_commands_under_root_dev.ymlfrom
05-11-move_react_native_dev_commands_under_root_dev.yml
Open

Move React Native dev commands under root dev.yml#64
kyle-schellen wants to merge 1 commit into
05-08-move_swift_dev_commands_under_root_dev.ymlfrom
05-11-move_react_native_dev_commands_under_root_dev.yml

Conversation

@kyle-schellen
Copy link
Copy Markdown

@kyle-schellen kyle-schellen commented May 11, 2026

Refs issue #908.

What changes are you making?

Move platforms/react-native/dev.yml into namespaced commands at the repo root so dev up and dev react-native <cmd> (or dev rn) work from the monorepo root. Last in the stack after #35 (Android) and #36 (Swift).

How to test

From the repo root:

  • dev up completes without error, including Node v22.14.0 / pnpm 10.33.1 setup, RN dependencies, gems, and pods
  • dev rn build builds the @shopify/checkout-sheet-kit module
  • dev rn lint runs SwiftLint/SwiftFormat checks, module lint, and sample lint
  • dev rn style still works as a legacy alias for lint
  • dev rn lint swift, dev rn lint module, and dev rn lint sample scope linting to one area
  • dev rn format runs RN Swift formatting
  • dev rn fix still works as a legacy alias for format
  • dev rn check runs license-header checks and lint
  • dev rn check license-headers checks RN module license headers
  • dev rn server starts Metro
  • dev rn s still works as a short alias for server
  • dev rn ios runs the iOS sample app
  • dev rn android runs the Android sample app
  • dev rn clean wipes module + sample workspaces and stops sccache

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.

Copy link
Copy Markdown
Author

kyle-schellen commented May 11, 2026

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

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

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 11, 2026

React Native — Coverage Report

Lines Statements Branches Functions
Coverage: 99%
99.53% (214/215) 95.96% (119/124) 100% (64/64)

@kyle-schellen kyle-schellen added the #gsd:50662 Rebase Checkout Kit on UCP label May 11, 2026 — with Graphite App
@kyle-schellen kyle-schellen self-assigned this May 11, 2026
@kyle-schellen kyle-schellen force-pushed the 05-11-move_react_native_dev_commands_under_root_dev.yml branch from 48f7c22 to e7c58f1 Compare May 11, 2026 22:25
@kyle-schellen kyle-schellen marked this pull request as ready for review May 11, 2026 22:25
@kyle-schellen kyle-schellen requested a review from a team as a code owner May 11, 2026 22:25
Comment thread dev.yml Outdated
Comment thread dev.yml Outdated
Comment thread dev.yml Outdated
name: Install pods (React Native sample)
met?: (cd platforms/react-native/sample/ios && bundle exec pod check --ignore-dev-pods)
meet: cd platforms/react-native && pnpm run pod-install
- custom:
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.

I think we might be able to remove this one too - now your improvement to the metro bundler has come in, which means we dont have to do the build to see changes, this is less relevant

@kyle-schellen kyle-schellen force-pushed the 05-11-move_react_native_dev_commands_under_root_dev.yml branch from e7c58f1 to 555e5d6 Compare May 12, 2026 13:58
@kyle-schellen kyle-schellen force-pushed the 05-08-move_swift_dev_commands_under_root_dev.yml branch 2 times, most recently from 94fc5d5 to 31fba85 Compare May 12, 2026 14:04
@kyle-schellen kyle-schellen force-pushed the 05-11-move_react_native_dev_commands_under_root_dev.yml branch 3 times, most recently from f9cf466 to 79b74a1 Compare May 12, 2026 16:37
@kyle-schellen kyle-schellen requested a review from a team as a code owner May 12, 2026 16:37
@kyle-schellen kyle-schellen force-pushed the 05-08-move_swift_dev_commands_under_root_dev.yml branch from 31fba85 to 92ff28b Compare May 12, 2026 16:37
@kyle-schellen kyle-schellen force-pushed the 05-11-move_react_native_dev_commands_under_root_dev.yml branch from 79b74a1 to fa6fa57 Compare May 12, 2026 17:41
@kyle-schellen kyle-schellen force-pushed the 05-08-move_swift_dev_commands_under_root_dev.yml branch 2 times, most recently from 8d11dc5 to d26f92e Compare May 12, 2026 18:22
@kyle-schellen kyle-schellen force-pushed the 05-11-move_react_native_dev_commands_under_root_dev.yml branch from fa6fa57 to 87da5c4 Compare May 12, 2026 18:22
@kyle-schellen kyle-schellen force-pushed the 05-08-move_swift_dev_commands_under_root_dev.yml branch from d26f92e to 3b360d1 Compare May 12, 2026 21:37
@kyle-schellen kyle-schellen force-pushed the 05-11-move_react_native_dev_commands_under_root_dev.yml branch from 87da5c4 to 3604836 Compare May 12, 2026 21:37
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.

2 participants