Skip to content

Strict license-header check unified across platforms#149

Draft
kiftio wants to merge 1 commit into
05-19-consolidate_license_header_workflowsfrom
05-19-strict_license-header_check_unified_across_platforms
Draft

Strict license-header check unified across platforms#149
kiftio wants to merge 1 commit into
05-19-consolidate_license_header_workflowsfrom
05-19-strict_license-header_check_unified_across_platforms

Conversation

@kiftio
Copy link
Copy Markdown
Contributor

@kiftio kiftio commented May 19, 2026

What changes are you making?

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 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
Contributor Author

kiftio commented May 19, 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.

@kiftio kiftio mentioned this pull request May 19, 2026
11 tasks
@github-actions
Copy link
Copy Markdown

Web — Coverage Report

Lines Statements Branches Functions
Coverage: 98%
98.96% (571/577) 90.41% (151/167) 100% (55/55)

@github-actions
Copy link
Copy Markdown

React Native — Coverage Report

Lines Statements Branches Functions
Coverage: 95%
95.52% (192/201) 92.3% (108/117) 100% (60/60)

Copy link
Copy Markdown
Contributor

kieran-osgood-shopify commented May 19, 2026

I think we were discussing removing the individual file license references in slack as other Shopify public repo's don't do this and just declare the root LICENSE

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants