-
-
Notifications
You must be signed in to change notification settings - Fork 335
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
Update dependency build_bazel_rules_apple to v3 #1673
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1673 +/- ##
==========================================
- Coverage 85.80% 85.77% -0.03%
==========================================
Files 566 566
Lines 27856 27856
==========================================
- Hits 23901 23894 -7
- Misses 3955 3962 +7 ☔ View full report in Codecov by Sentry. |
Size test resultOld size: 8674624 bytes |
d243c5a
to
cb9be34
Compare
5c5c32a
to
b2ce0a4
Compare
Bloaty Results 🐋Compared to main
Full report: https://maplibre-native.s3.eu-central-1.amazonaws.com/bloaty-results/pr-1673-compared-to-main.txtCompared to d387090 (legacy)
Full report: https://maplibre-native.s3.eu-central-1.amazonaws.com/bloaty-results/pr-1673-compared-to-legacy.txt |
@louwers, can this be merged? I'm not an expert in Bazel so I do not want to break anything 😊 |
086a64e
to
b8c2fe1
Compare
@ntadej I don't think so. The way we build the I hope this will be picked up by someone so we can use Bazel to do it: bazelbuild/rules_apple#2175 |
This PR contains the following updates:
2.5.0
->3.1.1
Release Notes
bazelbuild/rules_apple (build_bazel_rules_apple)
v3.1.1
Compare Source
What's Changed
This release is compatible with Bazel 6.x LTS, 7.x LTS, and 8.x rolling releases
MODULE.bazel Snippet
Workspace Snippet
v3.1.0
Compare Source
What's Changed
.swiftinterface
files to imported dynamic xcframeworkpublic_hdrs
by @brentleyjones in https://github.com/bazelbuild/rules_apple/pull/2266momc
under Xcode 15 by @jszumski in https://github.com/bazelbuild/rules_apple/pull/2276objc_generate_linkmap
toapple_verification_test
by @mattrobmattrob in https://github.com/bazelbuild/rules_apple/pull/2284xcarchives
by @maxwellE in https://github.com/bazelbuild/rules_apple/pull/2282bzl_library
dependencies by @brentleyjones in https://github.com/bazelbuild/rules_apple/pull/2292This release is compatible with Bazel 6.x LTS, 7.x LTS, and 8.x rolling releases
MODULE.bazel Snippet
Workspace Snippet
v3.0.0
Compare Source
What's Changed
Below are the changes that were also in previous release candidates.
What's Changed
.doccarchive
through a newdocc_archive
rule, thanks @luispadron! See the docs for an example--incompatible_enable_cc_toolchain_resolution
flip, thanks @comius!ios_xctestrun_runner
failing when using parallel testing with a false positive saying no tests ran, thanks @kapoorlakshya!device
constraintPotentially breaking changes
platform_mappings
file is now required in some cases (e.g. if you are on a Bazel version older than 7.0.0-pre.20230823.4). If you do not already have one you can copy rules_apple'sminimum_os_version
(#2124)bundle_name
differs fromname
. This allows having targets with the samebundle_name
in the sameBUILD
file, thanks @chiragramani!apple_common.multi_arch_split
with new transition, this should not be a breaking change but is required for bazel 7.x support. Please file any issues you find!--@​build_bazel_rules_apple//apple/build_settings:use_tree_artifacts_outputs
/--define=apple.experimental.tree_artifact_outputs=1
Notable additions
app_intents
attributeios_extension
,macos_extension
, andtvos_extension
rulesapple_xcframework
generated binariesNotable improvements
private_deps
when gathering resources for Apple rulesios_message_extension
, thanks @brentleyjones!Migrations
--@​build_bazel_rules_apple//apple/build_settings:use_tree_artifacts_outputs
should be used instead of--define=apple.experimental.tree_artifact_outputs=1
. Support for the define will be removed in the future.--@​build_bazel_rules_apple//apple/build_settings:signing_certificate_name
should be used instead of--ios_signing_cert_name
This release is compatible with 6.x LTS and bazel 7.x rolling releases
MODULE.bazel Snippet
Workspace Snippet
Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR has been generated by Mend Renovate. View repository job log here.