-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Merge release/19.6 (rc-2) #16285
Merge release/19.6 (rc-2) #16285
Conversation
…elease_1.73.1 Integrate gutenberg-mobile release 1.73.1
1. Moved the `notification_channel_*` strings back where they initially were in the `strings.xml` file (and thus under the appropriate `<!-- … -->` XML comment providing context about them). - The reason why they were moved around during merge in #16251 is because those strings, which come from the `stories-android` lib, were updated in the `stories-android` repo itself to have `tools:ignore="UnusedResources"` manually added to them - As a result, our tooling merging lib's `strings.xml` files saw those had different XML attributes and thus removed the old XML node then re-added the lib XML node at the end, as opposed to just update its text content (as it'd have otherwise just done were there no new attributes) - See Automattic/stories-android@d397e19 for where that was changed in `stories-android` 2. Manually added `tools:ignore="UnusedResources"` to all the strings that were added from the WordPressEditor module - We add that attribute to strings that come from being merged from libs, because the linter is not capable of detecting when those are used by a library vs the app. - Note that some of those might genuinely be unused (neither by the app… nor by the WordPressEditor module… nor by any of its transitive dependencies for which having those redefined here would be for the intent of overriding the ones from those transitive dependencies…), while others might be false positives (actually used… but it's just the linter failing to detect it) - In practice, we'd have expected that `checkDependencies = true` in `WordPress/build.gradle` would still allow lint to detect which of those strings from `WordPressEditor` might be used in that submodule even if not used by the app, and I think it actually does (because only 18 of those 51 strings from WPEditor generate an "UnusedResources" error) so we could have instead decided to only add the ignore attribute for those 18 ones but consider that `checkDependencies = true` is supposed to detect the rest. But we preferred to keep it simple and unblock ourselves (as this prevented me from building the app) and be consistent with how we do it for other libs rather than cherry-pick which ones to tag. We plan to revisit and audit all the strings to find the unused one at a later date as a cleanup project anyway. 3. Changed the exclusion regex from `values-.*/strings\.xml` to `values-.*/.*\.xml`, so that it not only ignores the `strings.xml` files from translated or variant resources (i.e. the ones ones other than `values/`), but also include other ones like `colors.xml`, because 2 of the Aztec colors were now detected as unused in the app (while they're likely used by the Aztec lib and we're just overriding them in the app)
You can trigger optional UI/connected tests for these changes by visiting CircleCI here. |
You can test the changes on this Pull Request by downloading the APKs: |
Found 1 violations: The PR caused the following dependency changes: \--- project :libs:editor:WordPressEditor
- \--- org.wordpress-mobile.gutenberg-mobile:react-native-gutenberg-bridge:v1.73.0
- +--- com.facebook.fresco:animated-gif:2.0.0
- | +--- com.parse.bolts:bolts-tasks:1.4.0
- | +--- com.facebook.soloader:soloader:0.6.0 -> 0.10.1
- | | +--- com.facebook.soloader:annotation:0.10.1
- | | \--- com.facebook.soloader:nativeloader:0.10.1
- | +--- com.facebook.fresco:fbcore:2.0.0 -> 2.5.0
- | \--- com.facebook.fresco:animated-base:2.0.0
- | +--- com.facebook.fresco:fbcore:2.0.0 -> 2.5.0
- | +--- com.facebook.fresco:imagepipeline-base:2.0.0 -> 2.5.0
- | | +--- com.facebook.infer.annotation:infer-annotation:0.18.0
- | | | +--- com.google.code.findbugs:jsr305:3.0.1 -> 3.0.2
- | | | \--- org.jetbrains.kotlin:kotlin-annotations-jvm:1.3.72
- | | +--- com.facebook.soloader:annotation:0.10.1
- | | +--- com.parse.bolts:bolts-tasks:1.4.0
- | | \--- com.facebook.fresco:fbcore:2.5.0
- | +--- com.facebook.fresco:imagepipeline:2.0.0 -> 2.5.0
- | | +--- com.facebook.soloader:nativeloader:0.10.1
- | | +--- com.facebook.soloader:annotation:0.10.1
- | | +--- com.parse.bolts:bolts-tasks:1.4.0
- | | +--- com.facebook.fresco:fbcore:2.5.0
- | | \--- com.facebook.fresco:imagepipeline-base:2.5.0 (*)
- | +--- com.facebook.fresco:animated-drawable:2.0.0
- | | +--- com.facebook.fresco:imagepipeline:2.0.0 -> 2.5.0 (*)
- | | +--- com.facebook.fresco:drawee:2.0.0 -> 2.5.0
- | | | +--- com.facebook.fresco:fbcore:2.5.0
- | | | +--- com.facebook.fresco:imagepipeline:2.5.0 (*)
- | | | +--- com.facebook.fresco:imagepipeline-native:2.5.0
- | | | | +--- com.facebook.fresco:imagepipeline:2.5.0 (*)
- | | | | +--- com.facebook.fresco:fbcore:2.5.0
- | | | | \--- com.facebook.soloader:soloader:0.10.1 (*)
- | | | +--- com.facebook.fresco:memory-type-ashmem:2.5.0
- | | | | +--- com.facebook.fresco:fbcore:2.5.0
- | | | | \--- com.facebook.fresco:imagepipeline:2.5.0 (*)
- | | | +--- com.facebook.fresco:memory-type-native:2.5.0
- | | | | +--- com.facebook.fresco:fbcore:2.5.0
- | | | | +--- com.facebook.fresco:imagepipeline:2.5.0 (*)
- | | | | +--- com.facebook.fresco:imagepipeline-native:2.5.0 (*)
- | | | | \--- com.facebook.soloader:nativeloader:0.10.1
- | | | +--- com.facebook.fresco:memory-type-java:2.5.0
- | | | | +--- com.facebook.fresco:fbcore:2.5.0
- | | | | +--- com.facebook.fresco:imagepipeline:2.5.0 (*)
- | | | | \--- com.facebook.fresco:imagepipeline-native:2.5.0 (*)
- | | | +--- com.facebook.fresco:ui-common:2.5.0
- | | | | \--- com.facebook.fresco:fbcore:2.5.0
- | | | \--- com.facebook.fresco:middleware:2.5.0
- | | | +--- com.facebook.fresco:fbcore:2.5.0
- | | | \--- com.facebook.fresco:ui-common:2.5.0 (*)
- | | \--- com.facebook.fresco:fbcore:2.0.0 -> 2.5.0
- | \--- com.parse.bolts:bolts-tasks:1.4.0
- +--- com.google.android.material:material:1.2.1 -> 1.6.0-alpha01 (*)
- +--- com.facebook.react:react-native:0.66.2
- | +--- com.facebook.infer.annotation:infer-annotation:0.18.0 (*)
- | +--- com.facebook.yoga:proguard-annotations:1.19.0
- | +--- javax.inject:javax.inject:1
- | +--- androidx.appcompat:appcompat:1.0.2 -> 1.3.1 (*)
- | +--- androidx.autofill:autofill:1.1.0 (*)
- | +--- androidx.swiperefreshlayout:swiperefreshlayout:1.0.0 -> 1.1.0 (*)
- | +--- com.facebook.fresco:fresco:2.5.0
- | | +--- com.facebook.fresco:soloader:2.5.0
- | | | +--- com.facebook.fresco:fbcore:2.5.0
- | | | \--- com.facebook.soloader:soloader:0.10.1 (*)
- | | +--- com.facebook.soloader:nativeloader:0.10.1
- | | +--- com.facebook.fresco:ui-common:2.5.0 (*)
- | | +--- com.facebook.fresco:fbcore:2.5.0
- | | +--- com.facebook.fresco:drawee:2.5.0 (*)
- | | +--- com.facebook.fresco:imagepipeline:2.5.0 (*)
- | | +--- com.facebook.fresco:imagepipeline-native:2.5.0 (*)
- | | +--- com.facebook.fresco:memory-type-ashmem:2.5.0 (*)
- | | +--- com.facebook.fresco:memory-type-native:2.5.0 (*)
- | | +--- com.facebook.fresco:memory-type-java:2.5.0 (*)
- | | +--- com.facebook.fresco:nativeimagefilters:2.5.0
- | | | +--- com.facebook.fresco:imagepipeline:2.5.0 (*)
- | | | +--- com.facebook.fresco:imagepipeline-native:2.5.0 (*)
- | | | +--- com.facebook.fresco:memory-type-ashmem:2.5.0 (*)
- | | | +--- com.facebook.fresco:memory-type-native:2.5.0 (*)
- | | | +--- com.facebook.fresco:memory-type-java:2.5.0 (*)
- | | | +--- com.facebook.soloader:nativeloader:0.10.1
- | | | +--- com.parse.bolts:bolts-tasks:1.4.0
- | | | \--- com.facebook.fresco:fbcore:2.5.0
- | | \--- com.facebook.fresco:nativeimagetranscoder:2.5.0
- | | +--- com.facebook.fresco:imagepipeline-base:2.5.0 (*)
- | | +--- com.facebook.soloader:nativeloader:0.10.1
- | | +--- com.parse.bolts:bolts-tasks:1.4.0
- | | \--- com.facebook.fresco:fbcore:2.5.0
- | +--- com.facebook.fresco:imagepipeline-okhttp3:2.5.0
- | | +--- com.facebook.fresco:fbcore:2.5.0
- | | +--- com.facebook.fresco:imagepipeline:2.5.0 (*)
- | | +--- com.facebook.fresco:imagepipeline-native:2.5.0 (*)
- | | +--- com.facebook.fresco:memory-type-ashmem:2.5.0 (*)
- | | +--- com.facebook.fresco:memory-type-native:2.5.0 (*)
- | | +--- com.facebook.fresco:memory-type-java:2.5.0 (*)
- | | \--- com.squareup.okhttp3:okhttp:3.12.1 -> 4.9.2 (*)
- | +--- com.facebook.fresco:ui-common:2.5.0 (*)
- | +--- com.facebook.soloader:soloader:0.10.1 (*)
- | +--- com.google.code.findbugs:jsr305:3.0.2
- | +--- com.squareup.okhttp3:okhttp:4.9.1 -> 4.9.2 (*)
- | +--- com.squareup.okhttp3:okhttp-urlconnection:4.9.1 -> 4.9.2 (*)
- | +--- com.squareup.okio:okio:2.9.0 (*)
- | \--- com.facebook.fbjni:fbjni-java-only:0.2.2
- | \--- com.facebook.soloader:nativeloader:0.10.1
- +--- org.wordpress-mobile:react-native-svg:9.13.6
- | \--- com.facebook.react:react-native:0.66.2 (*)
- +--- com.github.wordpress-mobile:react-native-video:5.2.0-wp-2
- | +--- com.facebook.react:react-native:0.66.2 (*)
- | +--- com.google.android.exoplayer:exoplayer:2.13.3
- | | +--- com.google.android.exoplayer:exoplayer-core:2.13.3
- | | | +--- androidx.annotation:annotation:1.1.0 -> 1.2.0
- | | | +--- com.google.android.exoplayer:exoplayer-common:2.13.3
- | | | | +--- androidx.annotation:annotation:1.1.0 -> 1.2.0
- | | | | \--- com.google.guava:guava:27.1-android
- | | | | +--- com.google.guava:failureaccess:1.0.1
- | | | | \--- com.google.guava:listenablefuture:9999.0-empty-to-avoid-conflict-with-guava
- | | | \--- com.google.android.exoplayer:exoplayer-extractor:2.13.3
- | | | +--- androidx.annotation:annotation:1.1.0 -> 1.2.0
- | | | \--- com.google.android.exoplayer:exoplayer-common:2.13.3 (*)
- | | +--- com.google.android.exoplayer:exoplayer-dash:2.13.3
- | | | +--- com.google.android.exoplayer:exoplayer-core:2.13.3 (*)
- | | | \--- androidx.annotation:annotation:1.1.0 -> 1.2.0
- | | +--- com.google.android.exoplayer:exoplayer-hls:2.13.3
- | | | +--- androidx.annotation:annotation:1.1.0 -> 1.2.0
- | | | \--- com.google.android.exoplayer:exoplayer-core:2.13.3 (*)
- | | +--- com.google.android.exoplayer:exoplayer-smoothstreaming:2.13.3
- | | | +--- com.google.android.exoplayer:exoplayer-core:2.13.3 (*)
- | | | \--- androidx.annotation:annotation:1.1.0 -> 1.2.0
- | | +--- com.google.android.exoplayer:exoplayer-transformer:2.13.3
- | | | +--- androidx.annotation:annotation:1.1.0 -> 1.2.0
- | | | \--- com.google.android.exoplayer:exoplayer-core:2.13.3 (*)
- | | \--- com.google.android.exoplayer:exoplayer-ui:2.13.3
- | | +--- com.google.android.exoplayer:exoplayer-core:2.13.3 (*)
- | | +--- androidx.annotation:annotation:1.1.0 -> 1.2.0
- | | +--- androidx.recyclerview:recyclerview:1.1.0 (*)
- | | \--- androidx.media:media:1.2.1 (*)
- | +--- androidx.annotation:annotation:1.1.0 -> 1.2.0
- | +--- androidx.core:core:1.1.0 -> 1.7.0 (*)
- | +--- androidx.media:media:1.1.0 -> 1.2.1 (*)
- | +--- com.google.android.exoplayer:extension-okhttp:2.13.3
- | | +--- com.google.android.exoplayer:exoplayer-common:2.13.3 (*)
- | | +--- androidx.annotation:annotation:1.1.0 -> 1.2.0
- | | \--- com.squareup.okhttp3:okhttp:3.12.11 -> 4.9.2 (*)
- | \--- com.squareup.okhttp3:okhttp:${OKHTTP_VERSION} -> 4.9.2 (*)
- +--- com.github.wordpress-mobile:react-native-linear-gradient:2.5.6-wp-2
- | \--- com.facebook.react:react-native:0.66.2 (*)
- +--- com.github.wordpress-mobile:react-native-slider:3.0.2-wp-2
- | \--- com.facebook.react:react-native:0.66.2 (*)
- +--- org.wordpress-mobile:react-native-get-random-values:1.4.0
- | \--- com.facebook.react:react-native:0.66.2 (*)
- +--- org.wordpress-mobile:react-native-masked-view:0.2.6
- | \--- com.facebook.react:react-native:0.66.2 (*)
- +--- com.github.wordpress-mobile:react-native-gesture-handler:2.2.0-wp-4
- | +--- com.github.wordpress-mobile:react-native-reanimated:2.4.1-wp-1
- | +--- com.facebook.react:react-native:0.66.2 (*)
- | +--- androidx.appcompat:appcompat:1.2.0 -> 1.3.1 (*)
- | +--- androidx.core:core-ktx:1.6.0 -> 1.7.0 (*)
- | \--- org.jetbrains.kotlin:kotlin-stdlib:1.5.20 -> 1.6.10 (*)
- +--- org.wordpress-mobile:react-native-screens:2.9.0
- | +--- com.facebook.react:react-native:0.66.2 (*)
- | +--- androidx.appcompat:appcompat:1.1.0 -> 1.3.1 (*)
- | +--- androidx.fragment:fragment:1.2.1 -> 1.3.6 (*)
- | +--- androidx.coordinatorlayout:coordinatorlayout:1.1.0 (*)
- | +--- androidx.swiperefreshlayout:swiperefreshlayout:1.0.0 -> 1.1.0 (*)
- | \--- com.google.android.material:material:1.1.0 -> 1.6.0-alpha01 (*)
- +--- org.wordpress-mobile:react-native-safe-area-context:3.2.0
- | \--- com.facebook.react:react-native:0.66.2 (*)
- +--- com.github.wordpress-mobile:react-native-reanimated:2.4.1-wp-1
- +--- com.github.wordpress-mobile:react-native-prompt-android:1.0.0-wp-2
- | +--- com.facebook.react:react-native:0.66.2 (*)
- | \--- androidx.appcompat:appcompat:1.0.0 -> 1.3.1 (*)
- +--- org.wordpress-mobile:react-native-webview:11.6.2
- | +--- com.facebook.react:react-native:0.66.2 (*)
- | \--- org.jetbrains.kotlin:kotlin-stdlib:1.4.10 -> 1.6.10 (*)
- +--- org.wordpress-mobile:react-native-clipboard:1.9.0
- | \--- com.facebook.react:react-native:0.66.2 (*)
- +--- org.wordpress-mobile.gutenberg-mobile:react-native-aztec:v1.73.0
- | +--- androidx.legacy:legacy-support-v4:1.0.0 (*)
- | +--- androidx.gridlayout:gridlayout:1.0.0 (*)
- | +--- androidx.cardview:cardview:1.0.0 (*)
- | +--- androidx.appcompat:appcompat:1.2.0 -> 1.3.1 (*)
- | +--- androidx.recyclerview:recyclerview:1.1.0 (*)
- | +--- com.facebook.react:react-native:0.66.2 (*)
- | +--- org.wordpress:aztec:v1.5.4 (*)
- | +--- org.wordpress.aztec:wordpress-shortcodes:v1.5.4 (*)
- | +--- org.wordpress.aztec:wordpress-comments:v1.5.4 (*)
- | +--- org.wordpress.aztec:glide-loader:v1.5.4
- | | +--- org.jetbrains.kotlin:kotlin-stdlib:1.4.20 -> 1.6.10 (*)
- | | +--- org.wordpress:aztec:v1.5.4 (*)
- | | \--- com.github.bumptech.glide:glide:4.10.0 -> 4.11.0 (*)
- | \--- org.jetbrains.kotlin:kotlin-stdlib-jdk8:1.5.32 (*)
- +--- org.jetbrains.kotlin:kotlin-stdlib-jdk8:1.5.32 (*)
- \--- org.wordpress-mobile:hermes-release-mirror:0.66.2
+ \--- org.wordpress-mobile.gutenberg-mobile:react-native-gutenberg-bridge:v1.73.1
+ +--- com.facebook.fresco:animated-gif:2.0.0
+ | +--- com.parse.bolts:bolts-tasks:1.4.0
+ | +--- com.facebook.soloader:soloader:0.6.0 -> 0.10.1
+ | | +--- com.facebook.soloader:annotation:0.10.1
+ | | \--- com.facebook.soloader:nativeloader:0.10.1
+ | +--- com.facebook.fresco:fbcore:2.0.0 -> 2.5.0
+ | \--- com.facebook.fresco:animated-base:2.0.0
+ | +--- com.facebook.fresco:fbcore:2.0.0 -> 2.5.0
+ | +--- com.facebook.fresco:imagepipeline-base:2.0.0 -> 2.5.0
+ | | +--- com.facebook.infer.annotation:infer-annotation:0.18.0
+ | | | +--- com.google.code.findbugs:jsr305:3.0.1 -> 3.0.2
+ | | | \--- org.jetbrains.kotlin:kotlin-annotations-jvm:1.3.72
+ | | +--- com.facebook.soloader:annotation:0.10.1
+ | | +--- com.parse.bolts:bolts-tasks:1.4.0
+ | | \--- com.facebook.fresco:fbcore:2.5.0
+ | +--- com.facebook.fresco:imagepipeline:2.0.0 -> 2.5.0
+ | | +--- com.facebook.soloader:nativeloader:0.10.1
+ | | +--- com.facebook.soloader:annotation:0.10.1
+ | | +--- com.parse.bolts:bolts-tasks:1.4.0
+ | | +--- com.facebook.fresco:fbcore:2.5.0
+ | | \--- com.facebook.fresco:imagepipeline-base:2.5.0 (*)
+ | +--- com.facebook.fresco:animated-drawable:2.0.0
+ | | +--- com.facebook.fresco:imagepipeline:2.0.0 -> 2.5.0 (*)
+ | | +--- com.facebook.fresco:drawee:2.0.0 -> 2.5.0
+ | | | +--- com.facebook.fresco:fbcore:2.5.0
+ | | | +--- com.facebook.fresco:imagepipeline:2.5.0 (*)
+ | | | +--- com.facebook.fresco:imagepipeline-native:2.5.0
+ | | | | +--- com.facebook.fresco:imagepipeline:2.5.0 (*)
+ | | | | +--- com.facebook.fresco:fbcore:2.5.0
+ | | | | \--- com.facebook.soloader:soloader:0.10.1 (*)
+ | | | +--- com.facebook.fresco:memory-type-ashmem:2.5.0
+ | | | | +--- com.facebook.fresco:fbcore:2.5.0
+ | | | | \--- com.facebook.fresco:imagepipeline:2.5.0 (*)
+ | | | +--- com.facebook.fresco:memory-type-native:2.5.0
+ | | | | +--- com.facebook.fresco:fbcore:2.5.0
+ | | | | +--- com.facebook.fresco:imagepipeline:2.5.0 (*)
+ | | | | +--- com.facebook.fresco:imagepipeline-native:2.5.0 (*)
+ | | | | \--- com.facebook.soloader:nativeloader:0.10.1
+ | | | +--- com.facebook.fresco:memory-type-java:2.5.0
+ | | | | +--- com.facebook.fresco:fbcore:2.5.0
+ | | | | +--- com.facebook.fresco:imagepipeline:2.5.0 (*)
+ | | | | \--- com.facebook.fresco:imagepipeline-native:2.5.0 (*)
+ | | | +--- com.facebook.fresco:ui-common:2.5.0
+ | | | | \--- com.facebook.fresco:fbcore:2.5.0
+ | | | \--- com.facebook.fresco:middleware:2.5.0
+ | | | +--- com.facebook.fresco:fbcore:2.5.0
+ | | | \--- com.facebook.fresco:ui-common:2.5.0 (*)
+ | | \--- com.facebook.fresco:fbcore:2.0.0 -> 2.5.0
+ | \--- com.parse.bolts:bolts-tasks:1.4.0
+ +--- com.google.android.material:material:1.2.1 -> 1.6.0-alpha01 (*)
+ +--- com.facebook.react:react-native:0.66.2
+ | +--- com.facebook.infer.annotation:infer-annotation:0.18.0 (*)
+ | +--- com.facebook.yoga:proguard-annotations:1.19.0
+ | +--- javax.inject:javax.inject:1
+ | +--- androidx.appcompat:appcompat:1.0.2 -> 1.3.1 (*)
+ | +--- androidx.autofill:autofill:1.1.0 (*)
+ | +--- androidx.swiperefreshlayout:swiperefreshlayout:1.0.0 -> 1.1.0 (*)
+ | +--- com.facebook.fresco:fresco:2.5.0
+ | | +--- com.facebook.fresco:soloader:2.5.0
+ | | | +--- com.facebook.fresco:fbcore:2.5.0
+ | | | \--- com.facebook.soloader:soloader:0.10.1 (*)
+ | | +--- com.facebook.soloader:nativeloader:0.10.1
+ | | +--- com.facebook.fresco:ui-common:2.5.0 (*)
+ | | +--- com.facebook.fresco:fbcore:2.5.0
+ | | +--- com.facebook.fresco:drawee:2.5.0 (*)
+ | | +--- com.facebook.fresco:imagepipeline:2.5.0 (*)
+ | | +--- com.facebook.fresco:imagepipeline-native:2.5.0 (*)
+ | | +--- com.facebook.fresco:memory-type-ashmem:2.5.0 (*)
+ | | +--- com.facebook.fresco:memory-type-native:2.5.0 (*)
+ | | +--- com.facebook.fresco:memory-type-java:2.5.0 (*)
+ | | +--- com.facebook.fresco:nativeimagefilters:2.5.0
+ | | | +--- com.facebook.fresco:imagepipeline:2.5.0 (*)
+ | | | +--- com.facebook.fresco:imagepipeline-native:2.5.0 (*)
+ | | | +--- com.facebook.fresco:memory-type-ashmem:2.5.0 (*)
+ | | | +--- com.facebook.fresco:memory-type-native:2.5.0 (*)
+ | | | +--- com.facebook.fresco:memory-type-java:2.5.0 (*)
+ | | | +--- com.facebook.soloader:nativeloader:0.10.1
+ | | | +--- com.parse.bolts:bolts-tasks:1.4.0
+ | | | \--- com.facebook.fresco:fbcore:2.5.0
+ | | \--- com.facebook.fresco:nativeimagetranscoder:2.5.0
+ | | +--- com.facebook.fresco:imagepipeline-base:2.5.0 (*)
+ | | +--- com.facebook.soloader:nativeloader:0.10.1
+ | | +--- com.parse.bolts:bolts-tasks:1.4.0
+ | | \--- com.facebook.fresco:fbcore:2.5.0
+ | +--- com.facebook.fresco:imagepipeline-okhttp3:2.5.0
+ | | +--- com.facebook.fresco:fbcore:2.5.0
+ | | +--- com.facebook.fresco:imagepipeline:2.5.0 (*)
+ | | +--- com.facebook.fresco:imagepipeline-native:2.5.0 (*)
+ | | +--- com.facebook.fresco:memory-type-ashmem:2.5.0 (*)
+ | | +--- com.facebook.fresco:memory-type-native:2.5.0 (*)
+ | | +--- com.facebook.fresco:memory-type-java:2.5.0 (*)
+ | | \--- com.squareup.okhttp3:okhttp:3.12.1 -> 4.9.2 (*)
+ | +--- com.facebook.fresco:ui-common:2.5.0 (*)
+ | +--- com.facebook.soloader:soloader:0.10.1 (*)
+ | +--- com.google.code.findbugs:jsr305:3.0.2
+ | +--- com.squareup.okhttp3:okhttp:4.9.1 -> 4.9.2 (*)
+ | +--- com.squareup.okhttp3:okhttp-urlconnection:4.9.1 -> 4.9.2 (*)
+ | +--- com.squareup.okio:okio:2.9.0 (*)
+ | \--- com.facebook.fbjni:fbjni-java-only:0.2.2
+ | \--- com.facebook.soloader:nativeloader:0.10.1
+ +--- org.wordpress-mobile:react-native-svg:9.13.6
+ | \--- com.facebook.react:react-native:0.66.2 (*)
+ +--- com.github.wordpress-mobile:react-native-video:5.2.0-wp-2
+ | +--- com.facebook.react:react-native:0.66.2 (*)
+ | +--- com.google.android.exoplayer:exoplayer:2.13.3
+ | | +--- com.google.android.exoplayer:exoplayer-core:2.13.3
+ | | | +--- androidx.annotation:annotation:1.1.0 -> 1.2.0
+ | | | +--- com.google.android.exoplayer:exoplayer-common:2.13.3
+ | | | | +--- androidx.annotation:annotation:1.1.0 -> 1.2.0
+ | | | | \--- com.google.guava:guava:27.1-android
+ | | | | +--- com.google.guava:failureaccess:1.0.1
+ | | | | \--- com.google.guava:listenablefuture:9999.0-empty-to-avoid-conflict-with-guava
+ | | | \--- com.google.android.exoplayer:exoplayer-extractor:2.13.3
+ | | | +--- androidx.annotation:annotation:1.1.0 -> 1.2.0
+ | | | \--- com.google.android.exoplayer:exoplayer-common:2.13.3 (*)
+ | | +--- com.google.android.exoplayer:exoplayer-dash:2.13.3
+ | | | +--- com.google.android.exoplayer:exoplayer-core:2.13.3 (*)
+ | | | \--- androidx.annotation:annotation:1.1.0 -> 1.2.0
+ | | +--- com.google.android.exoplayer:exoplayer-hls:2.13.3
+ | | | +--- androidx.annotation:annotation:1.1.0 -> 1.2.0
+ | | | \--- com.google.android.exoplayer:exoplayer-core:2.13.3 (*)
+ | | +--- com.google.android.exoplayer:exoplayer-smoothstreaming:2.13.3
+ | | | +--- com.google.android.exoplayer:exoplayer-core:2.13.3 (*)
+ | | | \--- androidx.annotation:annotation:1.1.0 -> 1.2.0
+ | | +--- com.google.android.exoplayer:exoplayer-transformer:2.13.3
+ | | | +--- androidx.annotation:annotation:1.1.0 -> 1.2.0
+ | | | \--- com.google.android.exoplayer:exoplayer-core:2.13.3 (*)
+ | | \--- com.google.android.exoplayer:exoplayer-ui:2.13.3
+ | | +--- com.google.android.exoplayer:exoplayer-core:2.13.3 (*)
+ | | +--- androidx.annotation:annotation:1.1.0 -> 1.2.0
+ | | +--- androidx.recyclerview:recyclerview:1.1.0 (*)
+ | | \--- androidx.media:media:1.2.1 (*)
+ | +--- androidx.annotation:annotation:1.1.0 -> 1.2.0
+ | +--- androidx.core:core:1.1.0 -> 1.7.0 (*)
+ | +--- androidx.media:media:1.1.0 -> 1.2.1 (*)
+ | +--- com.google.android.exoplayer:extension-okhttp:2.13.3
+ | | +--- com.google.android.exoplayer:exoplayer-common:2.13.3 (*)
+ | | +--- androidx.annotation:annotation:1.1.0 -> 1.2.0
+ | | \--- com.squareup.okhttp3:okhttp:3.12.11 -> 4.9.2 (*)
+ | \--- com.squareup.okhttp3:okhttp:${OKHTTP_VERSION} -> 4.9.2 (*)
+ +--- com.github.wordpress-mobile:react-native-linear-gradient:2.5.6-wp-2
+ | \--- com.facebook.react:react-native:0.66.2 (*)
+ +--- com.github.wordpress-mobile:react-native-slider:3.0.2-wp-2
+ | \--- com.facebook.react:react-native:0.66.2 (*)
+ +--- org.wordpress-mobile:react-native-get-random-values:1.4.0
+ | \--- com.facebook.react:react-native:0.66.2 (*)
+ +--- org.wordpress-mobile:react-native-masked-view:0.2.6
+ | \--- com.facebook.react:react-native:0.66.2 (*)
+ +--- com.github.wordpress-mobile:react-native-gesture-handler:2.2.0-wp-4
+ | +--- com.github.wordpress-mobile:react-native-reanimated:2.4.1-wp-1
+ | +--- com.facebook.react:react-native:0.66.2 (*)
+ | +--- androidx.appcompat:appcompat:1.2.0 -> 1.3.1 (*)
+ | +--- androidx.core:core-ktx:1.6.0 -> 1.7.0 (*)
+ | \--- org.jetbrains.kotlin:kotlin-stdlib:1.5.20 -> 1.6.10 (*)
+ +--- org.wordpress-mobile:react-native-screens:2.9.0
+ | +--- com.facebook.react:react-native:0.66.2 (*)
+ | +--- androidx.appcompat:appcompat:1.1.0 -> 1.3.1 (*)
+ | +--- androidx.fragment:fragment:1.2.1 -> 1.3.6 (*)
+ | +--- androidx.coordinatorlayout:coordinatorlayout:1.1.0 (*)
+ | +--- androidx.swiperefreshlayout:swiperefreshlayout:1.0.0 -> 1.1.0 (*)
+ | \--- com.google.android.material:material:1.1.0 -> 1.6.0-alpha01 (*)
+ +--- org.wordpress-mobile:react-native-safe-area-context:3.2.0
+ | \--- com.facebook.react:react-native:0.66.2 (*)
+ +--- com.github.wordpress-mobile:react-native-reanimated:2.4.1-wp-1
+ +--- com.github.wordpress-mobile:react-native-prompt-android:1.0.0-wp-2
+ | +--- com.facebook.react:react-native:0.66.2 (*)
+ | \--- androidx.appcompat:appcompat:1.0.0 -> 1.3.1 (*)
+ +--- org.wordpress-mobile:react-native-webview:11.6.2
+ | +--- com.facebook.react:react-native:0.66.2 (*)
+ | \--- org.jetbrains.kotlin:kotlin-stdlib:1.4.10 -> 1.6.10 (*)
+ +--- org.wordpress-mobile:react-native-clipboard:1.9.0
+ | \--- com.facebook.react:react-native:0.66.2 (*)
+ +--- org.wordpress-mobile.gutenberg-mobile:react-native-aztec:v1.73.1
+ | +--- androidx.legacy:legacy-support-v4:1.0.0 (*)
+ | +--- androidx.gridlayout:gridlayout:1.0.0 (*)
+ | +--- androidx.cardview:cardview:1.0.0 (*)
+ | +--- androidx.appcompat:appcompat:1.2.0 -> 1.3.1 (*)
+ | +--- androidx.recyclerview:recyclerview:1.1.0 (*)
+ | +--- com.facebook.react:react-native:0.66.2 (*)
+ | +--- org.wordpress:aztec:v1.5.4 (*)
+ | +--- org.wordpress.aztec:wordpress-shortcodes:v1.5.4 (*)
+ | +--- org.wordpress.aztec:wordpress-comments:v1.5.4 (*)
+ | +--- org.wordpress.aztec:glide-loader:v1.5.4
+ | | +--- org.jetbrains.kotlin:kotlin-stdlib:1.4.20 -> 1.6.10 (*)
+ | | +--- org.wordpress:aztec:v1.5.4 (*)
+ | | \--- com.github.bumptech.glide:glide:4.10.0 -> 4.11.0 (*)
+ | \--- org.jetbrains.kotlin:kotlin-stdlib-jdk8:1.5.32 (*)
+ +--- org.jetbrains.kotlin:kotlin-stdlib-jdk8:1.5.32 (*)
+ \--- org.wordpress-mobile:hermes-release-mirror:0.66.2
Please review and act accordingly
|
<string name="notification_channel_reminder_id" translatable="false" tools:ignore="UnusedResources">wpandroid_notification_reminder_channel_id</string> | ||
|
||
<!-- WordPressEditor module --> | ||
<string name="post_content" tools:ignore="UnusedResources">Content</string> |
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.
This one and all the ones below are because we fixed the linter error by adding tools:ignore="UnusedResources"
in the original WordPress/src/main/res/values/strings.xml
, so since that strings.xml
changed, we is why that frozen fastlane/resources/values/strings.xml
got then updated accordingly to apply the same changes made during release/19.6
to the frozen ones.
That should not impact the GlotPress import, since that change in the frozen strings is only about moving some nodes around, and adding the tools:ignore
attribute, but didn't change their name
nor value/text; so I don't thing GlotPress will see when re-importing the file next time around
@@ -4010,6 +4010,59 @@ translators: %s: Select control option value e.g: "Auto, 25%". --> | |||
<string name="blogging_prompts_onboarding_remind_me">Remind me</string> | |||
<string name="blogging_prompts_onboarding_card_prompt">Cast the movie of your life.</string> | |||
|
|||
<!-- WordPressEditor module --> |
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.
Adding the tools:ignore="UnusedResources"
here (as well as that comment) to all the entries that got previously merged from the WordPressEditor module, so that the linter doesn't error on those strings (not realizing they might not be used by the main/src/
code but might still be used by the subproject)
@@ -81,7 +81,7 @@ | |||
<ignore path="**/WordPressEditor/**" /> | |||
<ignore path="**/.gradle/caches/**" /> | |||
<ignore path="**/generated/**" /> | |||
<ignore regexp=".*\/values\-([a-z]|[A-Z]|\-){2,6}\/strings\.xml$"/> | |||
<ignore regexp=".*\/values\-([a-z]|[A-Z]|\-){2,6}\/.*\.xml$"/> |
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.
I think this is fine, but I would have preferred to be more verbose and add each file separately - assuming it's less than 5 or so files of course. My slight worry is that we'll miss something due to this catch all file name pattern. However, I can't find a concrete example where it'll be an issue, so I am just leaving this as a note.
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.
Good point, addressed in #16293
By annotating the 2 problematic color resources individually, instead of adjusting the exception rule with a too-broad regexp See https://github.com/wordpress-mobile/WordPress-Android/pull/16285\#discussion_r846318394
Merging the release branch after doing a new beta.
The new beta includes:
strings.xml
from GlotPressversion.properties
tools:ignore="UnusedResources"
) for some linter errors that started to appear on my machineConflicts Resolution
The conflicts were due to the changes made to fix the tooling and the linter errors on UnusedResources for strings merged from WordPressEditor, while some new strings were introduced in
trunk
in the meantime:1st conflict: notification_channel_* entries moved
2nd conflict: tools:ignore vs new Initial Screen strings
I solved them by:
release/19.6
for thenotification_channel_*
strings on the first conflict, since those were made in this branch to fix the glitch from recent lib merge during code freeze.release/19.6
resulted in a conflict withtrunk
is because of a tab-vs-space change (likely done automatically by some devs' editors) intrunk
<!-- Initial Screen -->
sectiontrunk
by @zwarm during My Site Dashboard: Initial Screen App Setting #16252trunk
release/19.6
which added thetools:ignore="UnusedResources"
attributes to the WordPressEditor module' strings