Zac Sweers

Android 10 leaks Activity#mActivityTransitionState

2

https://issuetracker.google.com/issues/141132765 TL;DR - Every shared element transition leaks the view that was used in the transition, which means at least that view is leaked after detach and likely also that...

[Lint] Lint should not depend on compilation

4

It runs on ASTs, so it can run in parallel to compilation. As a potential followup, we can maybe look at making lint optionally run as an implicit concurrent action...

Suggestion: Accept a `Nullable` annotation name and suggest it in cases where it's missing

1

Error-prone itself has some checks for this, but nullaway is obviously better at it. Given an error like this ``` /Users/zsweers/dev/slack/restructure/slack-android-ng/libraries/foundation/slack-commons/src/main/java/slack/commons/logger/DebugLogger.java:62: error: [NullAway] parameter tag is @NonNull, but parameter in...

Feature request: Recognize common "assertThat(...).isNotNull()" statements

2

These are common in tests ```java assertThat(foo).isNotNull(); foo.bar(); //

Improve the Java language color + match modern branding

15

The previous color no longer matches the Java branding colors and hasn't aged well. This matches the newer branding Before: `#b07219` After: `#2a6277`  ## Checklist: - [ ] **I...

Ref: #370

Context receiver APIs

4

If [compound extensions](https://discuss.kotlinlang.org/t/compound-extension/10722/1) do make their way into Kotlin, we could leverage them nicely in AutoDispose to do something like this: ```kotlin fun ScopeProvider.Observable.autoDispose() { // ... } fun test(scope:...

Gives us a lot more coverage and ensures they're kept at parity, as well as avoiding duplication.

Resolves #39 Resolves #97 (hopefully) Resolves #59 (hopefully) Very much a work in progress. This replaces the current StorageBackedService approach with androidx.paging's solution instead. Part of this will require more...