Merge "Report UnflaggedApi issues as warnings but treat as errors when new" into main am: 5fa409b88e
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/3248013 Change-Id: Ieeb57fcf4365e1134123f17a16a7ad8d6d5d6bcd Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
@@ -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" {
|
||||||
|
Reference in New Issue
Block a user