Add linting for unsorted-dict-items (#89)

Co-authored-by: Alex Eagle <[email protected]>

Author: jklukas

.github/workflows/BUILD.bazel

@@ -2,6 +2,7 @@ load("@buildifier_prebuilt//:rules.bzl", "buildifier")
 
 buildifier(
     name = "buildifier.check",
+    args = ["--warnings=unsorted-dict-items"],
     exclude_patterns = ["./.git/*"],
     lint_mode = "warn",
     mode = "diff",

.pre-commit-config.yaml

@@ -18,6 +18,7 @@ repos:
     hooks:
       - id: buildifier
       - id: buildifier-lint
+        args: ["-warnings=unsorted-dict-items"]
   # Enforce that commit messages allow for later changelog generation
   - repo: https://github.com/commitizen-tools/commitizen
     rev: v2.18.0

protoc/extensions.bzl

@@ -34,6 +34,7 @@ def _proto_extension_impl(module_ctx):
 protoc = module_extension(
     implementation = _proto_extension_impl,
     tag_classes = {
+        # buildifier: disable=unsorted-dict-items
         "toolchain": tag_class(attrs = {
             "name": attr.string(doc = """\
                 Base name for generated repositories, allowing more than one toolchain to be registered.