From 17b8b6f804e4b4e27ca538c872a59d02e1aa9e98 Mon Sep 17 00:00:00 2001 From: Andres Suarez Date: Mon, 10 Jul 2023 16:20:24 -0700 Subject: [PATCH] [lint] Correctly suppress BUILDIFIERLINT Summary: `lint-ignore` and `lint-ignore-every` bluntly silence any issue, whereas buildifier suppression directives allow for precise narrow per-issue suppression. So replace them. Test Plan: `arc lint` comes back clean Reviewed By: dtolnay Differential Revision: D47348679 fbshipit-source-id: a56c0ee97ac1a9066094b7dfff3bff6a203b5d94 --- antlir/bzl/image/feature/install.bzl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/antlir/bzl/image/feature/install.bzl b/antlir/bzl/image/feature/install.bzl index 47fbfa94099..1ce9e8da5d9 100644 --- a/antlir/bzl/image/feature/install.bzl +++ b/antlir/bzl/image/feature/install.bzl @@ -199,7 +199,7 @@ def feature_install( user = "root", group = "root", separate_debug_symbols = True, - # @lint-ignore BUILDIFIERLINT + # buildifier: disable=unused-variable wrap_as_buck_runnable = False): """ `feature.install("//path/fs:data", "dir/bar")` installs file or directory