StrixOS/build_soong
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Report UnflaggedApi issues as warnings but treat as errors when new" into main

Browse Source
This commit is contained in:
Treehugger Robot
2024-08-28 19:13:24 +00:00
committed by Gerrit Code Review
parent b41b74fa2c 88d3b39846
commit 5fa409b88e
1 changed files with 5 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
java/droidstubs.go
View File

@@ -970,6 +970,11 @@ func (d *Droidstubs) everythingOptionalCmd(ctx android.ModuleContext, cmd *andro
d.apiLintReport = android.PathForModuleOut(ctx, Everything.String(), "api_lint_report.txt") d.apiLintReport = android.PathForModuleOut(ctx, Everything.String(), "api_lint_report.txt")
cmd.FlagWithOutput("--report-even-if-suppressed ", d.apiLintReport) // TODO: Change to ":api-lint" cmd.FlagWithOutput("--report-even-if-suppressed ", d.apiLintReport) // TODO: Change to ":api-lint"
// Make sure that existing UnflaggedApi issues are reported as warnings but issues in
// new/changed code are treated as errors by the Build Warnings Aye Aye Analyzer in Gerrit.
// Once existing issues have been fixed this will be changed to error.
cmd.Flag("--error-when-new UnflaggedApi")
// TODO(b/154317059): Clean up this allowlist by baselining and/or checking in last-released. // TODO(b/154317059): Clean up this allowlist by baselining and/or checking in last-released.
if d.Name() != "android.car-system-stubs-docs" && if d.Name() != "android.car-system-stubs-docs" &&
d.Name() != "android.car-stubs-docs" { d.Name() != "android.car-stubs-docs" {