Skip to content
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

IOS-8413: [Tech] Use prebuilt swift-protobuf-binaries dependency (FB 10.29.0) #1

Merged

Conversation

m3g0byt3
Copy link

@m3g0byt3 m3g0byt3 commented Nov 27, 2024

В SDK есть одна зависимость (btw мы ее не юзаем) FirebaseMLModelDownloader, которая зависит от SwiftProtobuf:

.package(
  url: "https://github.com/apple/swift-protobuf.git",
  "1.19.0" ..< "2.0.0"
),

Поэтому пришлось сделать правку, чтобы юзались бинарники протобафа


Для обновления на новые версии FB я предлагаю такую схему:

  1. Синкаем апстрим кнопкой в gh
  2. Выбираем версию, на которую хотим обновить FB - например последнюю на сегодня 11.6.0
  3. Отводим от этого тега ветку вида release-11.6.0 (для некоторых релизов такие ветки уже есть, видимо в апстриме делались какие-то правки)
  4. В release-11.6.0 делаем пулл из фича-ветки, в которую пикнут один единственный коммит 5faa3aec04c2dda27c1e33196a9559a2ecdd4659 (с правкой для SwiftProtobuf)
  5. После мерджа фича-ветки в release-11.6.0 вешаем на коммит мержа тег 11.6.0-tangem1, на который будет смотреть приложение

Таким образом будут исключены конфликты при подливании апстрима

@m3g0byt3 m3g0byt3 changed the title IOS-8413: Use prebuilt swift-protobuf-binaries dependency (FB 10.29.0) IOS-8413: [Tech] Use prebuilt swift-protobuf-binaries dependency (FB 10.29.0) Nov 27, 2024
@tureck1y
Copy link
Collaborator

Сделай обязательно страничку в ноушене с инструкцией, в иосной папке

@m3g0byt3 m3g0byt3 merged commit 8052608 into release-10.29.0 Nov 28, 2024
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.

3 participants