Fixed
Status Update
Comments
hu...@google.com <hu...@google.com>
je...@google.com <je...@google.com>
al...@google.com <al...@google.com>
al...@google.com <al...@google.com>
al...@google.com <al...@google.com>
se...@team.bumble.com <se...@team.bumble.com> #2
This was broken by 3bca759a5ff08352de831bb1e9b61b1ec2b3362d.
Fix (pending) is I2c2dc7b600603ee430fd0d91b23d52ea8aa29ca9.
Fix (pending) is I2c2dc7b600603ee430fd0d91b23d52ea8aa29ca9.
ki...@gmail.com <ki...@gmail.com> #3
Almost 2 months later and this is still broken
ki...@gmail.com <ki...@gmail.com> #4
Since there is no progression, I wanted to share our quick-fix for the issue.
#sdkmanager --package_file=${PATH_WORKSPACE}/packages
while read p; do echo "y" | sdkmanager "${p}"; done <${PATH_WORKSPACE}/packages
#sdkmanager --package_file=${PATH_WORKSPACE}/packages
while read p; do echo "y" | sdkmanager "${p}"; done <${PATH_WORKSPACE}/packages
Description
Related to https://issuetracker.google.com/issues/263876380 , I am trying to configure Checkstyle and Detekt static analysis tools to work with the new issue 263876380 , I am using the
gradle-api
AGP artifact. PerVariant#sources.java.all
API to get the files to pass to Checkstyle. I am noticing that this API includes all sources that are generated by tasks registered as code generation tasks, which makes sense for the API in general, but does not work for static analysis tools like Checkstyle or Detekt.Ideally we need an API that will give us only the source files that are actually in the repo and human-editable. Scanning generated files is not useful because developers usually can't edit them to fix violations, and these tools expect you to not pass the files to the tool in the first place and don't provide great options for filtering them out after the fact.
Last tested with AGP 8.1.0-alpha01