-
Notifications
You must be signed in to change notification settings - Fork 295
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Sync repo in an SPM-friendly way #3827
Changes from all commits
426240f
1e7c238
0551b7e
7763170
94eacb3
de211b6
a22ca6e
b570cc4
9c29261
447af08
4b8c573
e35dd84
9d7f5a2
e71a789
0483ebf
2f4e6f2
c343d1c
739a127
46346d9
e5c5f89
f24b987
abede36
df3653e
3f14666
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -339,6 +339,20 @@ commands: | |
path: fastlane/test_output/xctest | ||
destination: scan-test-output | ||
|
||
push-to-spm: | ||
steps: | ||
- checkout | ||
- setup-git-credentials | ||
- run: | ||
name: Clone purchases-ios and push to purchases-ios-spm | ||
command: | | ||
git clone https://github.com/RevenueCat/purchases-ios.git | ||
cd purchases-ios | ||
git fetch --tags | ||
git remote set-url origin https://github.com/RevenueCat/purchases-ios-spm.git | ||
git push origin | ||
git push --tags | ||
Comment on lines
+349
to
+354
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. [not needed for this PR] ideally we'd move this to fastlane, we try to keep the CI stuff as simple as possible and move stuff into fastlane which is much easier to test. |
||
|
||
jobs: | ||
spm-release-build-xcode-14: | ||
<<: *base-job | ||
|
@@ -1007,6 +1021,7 @@ jobs: | |
- run: | ||
name: Create automatic PR | ||
command: bundle exec fastlane automatic_bump github_rate_limit:10 | ||
- push-to-spm | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. is there a reason why this is done in the The Is it intentional? |
||
|
||
integration-tests-all: | ||
<<: *base-job | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this will always be running from a clone, right? why do we do this instead of adding the other repo as an alt remote to the existing repo?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As discussed, merging now because it's working and we are biased to shipping, will return to this as a possible future improvement.