Skip to content

Commit 95336a9

Browse files
authored
fix(ci): update dep workflow usage (#492)
* Update * Update * Update * Update * Update * Update * Update * Update update-deps.yml to remove pull_request trigger Remove temporary pull_request trigger from workflow
1 parent 38b606b commit 95336a9

File tree

2 files changed

+7
-4
lines changed

2 files changed

+7
-4
lines changed

.github/workflows/update-deps.yml

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,25 +1,24 @@
11
name: Update Dependencies
22

33
on:
4-
# Run every day.
54
schedule:
65
- cron: '0 3 * * *'
7-
# And on on every PR merge so we get the updated dependencies ASAP, and to make sure the changelog doesn't conflict.gi
86
push:
97
branches:
108
- main
119

1210
jobs:
1311
cocoa:
14-
uses: getsentry/github-workflows/.github/workflows/updater.yml@v3
12+
uses: getsentry/github-workflows/.github/workflows/updater.yml@v2
1513
with:
1614
path: scripts/update-cocoa.sh
1715
name: Cocoa SDK
16+
runs-on: macos-latest
1817
secrets:
1918
api-token: ${{ secrets.CI_DEPLOY_KEY }}
2019

2120
java:
22-
uses: getsentry/github-workflows/.github/workflows/updater.yml@v3
21+
uses: getsentry/github-workflows/.github/workflows/updater.yml@v2
2322
with:
2423
path: scripts/update-java.sh
2524
name: Java SDK

scripts/update-cocoa.sh

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -75,6 +75,10 @@ set-version)
7575
newValue="${sample_podspec_var_name}'$2'"
7676
echo "${sample_podspec_content/${sample_podspec_whole_match}/$newValue}" >$sample_podspec_file
7777

78+
# Generate dummy framework first (required for pod update to work)
79+
echo "Generating dummy framework for shared module..."
80+
./gradlew :sentry-samples:kmp-app-cocoapods:shared:generateDummyFramework
81+
7882
# Run pod update in the sample iOS app directory to update Podfile.lock
7983
echo "Running pod update in $sample_ios_app_dir..."
8084
(cd $sample_ios_app_dir && pod update)

0 commit comments

Comments
 (0)