ankidroid / ankidroid/Anki-Android

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

オープン
#21,500 コメント 3 件 リアクション 0 件 担当者 1 名 @david-allison が担当を希望しています GitHub で見る
主要言語
Kotlin
スター
11.8k
フォーク
2.9k
平均マージ
2日 3時間
マージ済み PR(30日)
171

説明

> **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)_

コントリビューションガイド

コントリビューションガイドを開く

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。