Align workspace dev commands in root dev.yml#69
Conversation
7077869 to
c4188d2
Compare
c4188d2 to
cf8541b
Compare
f9cf466 to
79b74a1
Compare
79b74a1 to
fa6fa57
Compare
cf27458 to
ea11c12
Compare
fa6fa57 to
87da5c4
Compare
| supported_exts = %w[.swift .h .mm .java .js .ts .tsx] | ||
|
|
||
| Find.find(dir) do |path| | ||
| next if path.include?('/build/generated/') |
There was a problem hiding this comment.
This keeps the RN license check focused on source-controlled files. Generated Android build output can exist locally after builds/codegen and does not reliably include our license header, which made dev check depend on local build state
| @@ -292,7 +292,7 @@ class RCTShopifyCheckoutSheetKit: RCTEventEmitter, CheckoutDelegate { | |||
| return NSNumber(value: available) | |||
| } | |||
|
|
|||
| @objc func initiateGeolocationRequest(_ allow: Bool) { | |||
| @objc func initiateGeolocationRequest(_: Bool) { | |||
There was a problem hiding this comment.
Root dev check surfaced an existing SwiftFormat issue. The parameter is intentionally unused, so _ is the correct shape here.
ea11c12 to
f46976e
Compare
87da5c4 to
3604836
Compare
9514af8 to
de4dcf6
Compare
|
|
||
| ## Dev workflow | ||
|
|
||
| Run `dev` commands from the repo root. Use `dev up` before running commands when |
There was a problem hiding this comment.
Not sure if you've had success with this locally but I think telling it to use shadoenv and where to get dev from helped with certain ai sandboxing
All commands require `shadowenv exec --` prefix.
shadowenv exec --dir DIR -- /opt/dev/bin/dev up
shadowenv exec --dir DIR -- /opt/dev/bin/dev test [ARGS]
| - [Privacy compliance](https://shopify.dev/docs/storefronts/mobile/checkout-kit/privacy-compliance) - pass GDPR, CCPA, and ATT consent through to Shopify. | ||
| - [Accelerated checkouts](https://shopify.dev/docs/storefronts/mobile/checkout-kit/accelerated-checkouts?extension=react-native) - Shop Pay and Apple Pay buttons for one-tap purchase on product and cart pages. | ||
|
|
||
| ## Development |
There was a problem hiding this comment.
I think we can move this section into the CONTRIBUTING.md to help keep the readme slim
de4dcf6 to
d4ac612
Compare
3604836 to
14080fe
Compare
85710fe to
8496e6e
Compare
|
@kieran-osgood-shopify Thanks for the review. I did another pass now that the lower stack merged in. I cleaned up the root |
| react-native-lint-module: cd platforms/react-native && pnpm module lint | ||
| react-native-lint-sample: cd platforms/react-native && pnpm sample lint | ||
| react-native-license-headers: cd platforms/react-native && ./scripts/copy_license --check | ||
| protocol-build: swift build --target ShopifyCheckoutProtocol |
There was a problem hiding this comment.
I believe dev check is mainly for lint checks (supposed to finish fast) so a build might be a bit heavy for check
8496e6e to
c4e0c6b
Compare
1bf5d08 to
8575e24
Compare
9580dc7 to
d73ca4c
Compare
d73ca4c to
3028a16
Compare
3028a16 to
d3d925e
Compare
d3d925e to
13564e3
Compare
Closes shop/issues-checkout-kit#908.
This bottom PR establishes the root
dev.ymlcommand shape for Checkout Kit. It adds repo-wide aggregates for Android, Swift, React Native, and Web; keeps protocol available throughdev protocol <command>; standardizesformat/fixandlint/style; adds thedev rnalias; and makes bare API commands print usage and fail when a required subcommand is missing.Reviewers should focus on the command surface and whether the platform-scoped commands remain usable in isolation. Verified locally with
dev.ymlparsing, representativedev helpoutput, API usage commands, and stack diff checks.How to test
The bare API commands should print usage and exit non-zero.