Buildifier fix has surprising behavior when an unused load statement is found on the top of the file

[yashhshah]

buildifier version: 7.3.1

Description

Buildifier is incorrectly removing comments from the top of the build file while trying to fix an unused load statement at the top of the file.

An example BUILD file would have something like:

# some comment
load("//:target.bzl", "unused")

When removing the unused load statement, buildifier also removes the comment above it.

[laurentlb]

The recommendation is to put an empty line between the comment and the load statement.

Without separating line, the comment is "attached" to the next statement. Buildifier assumes the comment is for the load itself.