Fixed
Status Update
Comments
ap...@google.com <ap...@google.com> #2
Nested scrolling works partially (as per http://b/122818889 ). Let's discuss if we need full support and if so make sure it works.
jb...@google.com <jb...@google.com> #3
Hi!
What is 'partially' exactly?
How do I see it?
Thanks!
What is 'partially' exactly?
How do I see it?
Thanks!
Description
navigation-lint-common
hosts test helpers that generatesTestFile
.However, b/368070326 . Instead, it should be in test-specific source sets.
TestFile
is for test only and should not reside in production code. It leads toWe should refactor lint-common code into a new structure with common and common-test submodules
This was done in these CLs aosp/3284996 . But this fix is blocked by recent bytecode migration in ag/29027860 . During migration, the existing path workaround , so this new path breaks presubmit. Thread for context.
"navigation:navigation-lint-common"
was hardcoded into aWhile waiting for an updated workaround or a patch from lint infra, we should in the meantime copy paste
TestFile
related code into each lint test artifact so that we restore lint in the mean time.