-
Notifications
You must be signed in to change notification settings - Fork 3
Implement GutenbergKitResources binary dependency build #318
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
Changes from all commits
7332964
9e26d95
cbb366d
b928a23
6f5cd7b
f527e1d
b010604
bec2550
c847845
a48f2fe
f9a7ed7
ea5780e
be98603
46d666f
6052b98
25ad376
e223fa9
5d4b5ae
9b3a47c
fa64d04
af9ecd9
55cffbb
12f8625
7ec3efa
b6351de
ff312e5
be6c3cf
0735fb6
d8b578a
922cbbc
0519f23
629afc6
c25d6b3
afb078a
769104f
9d10b90
9faf342
f433dee
a34c0f6
a750758
91e3d1d
3f0ed64
6e11b25
3fc854d
6ef0dcc
0a56c19
4ab8263
776f79e
db62103
a5b855e
d281fcc
33b4d8d
defdaa8
15075b5
690a76f
8070810
0184d1b
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 |
|---|---|---|
| @@ -0,0 +1,22 @@ | ||
| #!/usr/bin/env bash | ||
|
|
||
| set -euo pipefail | ||
|
|
||
| version="$1" | ||
|
|
||
| if [[ -z "$version" ]]; then | ||
| echo "Error: version argument is required" | ||
| exit 1 | ||
| fi | ||
|
|
||
| echo "--- :robot_face: Use bot for Git operations" | ||
| source use-bot-for-git | ||
|
|
||
| echo "--- :rubygems: Installing gems" | ||
| install_gems | ||
|
|
||
| echo "--- :xcode: Building XCFramework" | ||
| make build-resources-xcframework REFRESH_L10N=1 REFRESH_JS_BUILD=1 | ||
|
Member
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. Similar to my comment on the Buildkite configuration, should/can we reuse the JavaScript build artifact?
Member
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. Struggling to understand how these XCFramework releases coincide/integrate with GutenbergKit releases. The existing release script for the latter bumps the version for inclusion in the JavaScript, so that accurate version numbers are reported in exceptions. Should the XCFramework and GutenbergKit versions align? Maybe that is only required for tagged GutenbergKit releases, and XCFramework releases for Git commits can simply use the last tagged version string? I had Claude capture the issue and potential solution in its own words below as well. I'm remain uncertain as to the current state, plan, or appropriate solution. WDYT?
Contributor
Author
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. Thanks for this @dcalhoun . To be honest, I'm still unsure about how to proceed, too. So far, I've just tried to mimic the wordpress-rs behavior to have CI track the checksum and revision in the You bring up a good point regarding the version bump. It's frustrating that there's no way to do a version bump and I'll have to think about this... |
||
|
|
||
| echo "--- :rocket: Publishing release $version" | ||
| bundle exec fastlane release "version:$version" | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,4 @@ | ||
| --- | ||
| BUNDLE_PATH: "vendor/bundle" | ||
| BUNDLE_SPECIFIC_PLATFORM: "false" | ||
| BUNDLE_FORCE_RUBY_PLATFORM: "true" |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,6 @@ | ||
| # frozen_string_literal: true | ||
|
|
||
| source 'https://rubygems.org' | ||
|
|
||
| gem 'fastlane', '~> 2.230' | ||
| gem 'fastlane-plugin-wpmreleasetoolkit', '~> 13.8' |
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.
After #320, we now cache the JavaScript build as an artifact to improve speed and mitigate 429 errors for translation downloads. Can we reuse that artifact for this 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.
For sure!
https://linear.app/a8c/issue/AINFRA-2159/resue-cached-js-in-ci