ankidroid / ankidroid/Anki-Android

multimodule support: move strings to `:common:android`

Đang mở
#21,500 3 bình luận 0 reaction 1 người được giao Được @david-allison nhận Xem trên GitHub
Ngôn ngữ chính
Kotlin
Star
11.8k
Fork
2.9k
Merge trung bình
2 ngày 3 giờ
Pull request đã merge (30 ngày)
171

Mô tả

> **Strings**: move all into `:common:android`.
>
> This unblocks us simply/quickly and allows a future discussion about the CrowdIn sync/file architecture.
>
> ----
>
> Medium-term: I think `:widgets` containing strings will be a "Won't fix" for a while
>
> Architecturally, strings should live in their modules, but I think moving strings up into `:widgets` would:
> * Mildly improve `R.string` via namespacing
> * Produce minimal performance gains (the resources would still need to be bundled inside the app).
> * Not improve discoverability (developers know where the strings are)
> * Add disruption/infra complexity in the case that a string needs to be pulled up/down from `:widgets` into `common:android`
> * Also with lint & cross-module duplication checks
> * Not have any tangible benefit to translators
>
> I don't see the upsides being worth the time to update our translation infra for this.

_Originally posted by @david-allison in [#20737](https://github.com/ankidroid/Anki-Android/issues/20737#issuecomment-4303120972)_

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Đánh giá

Issue này chưa được đánh giá.

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.