helm: tidy BUILD files, move rules to helm subdirectory

BUILD

@@ -24,31 +24,3 @@ sh_test(
         ".dummy_test.sh",
     ],
 )
-
-load("@io_bazel_skydoc//stardoc:stardoc.bzl", "stardoc")
-load("@bazel_skylib//:bzl_library.bzl", "bzl_library")
-
-# package(default_visibility = ["//visibility:public"])
-package(default_visibility = ["//visibility:public"])
-
-load("@bazel_tools//tools/build_defs/pkg:pkg.bzl", "pkg_tar")
-
-bzl_library(
-
-    name = "skylib_paths",
-    srcs = ["@bazel_skylib//lib:paths.bzl"],
-)
-# bzl_library(
-#     name = "bazel_tools_pkg",
-#     srcs = ["@bazel_tools//tools:srcs"],
-# )
-
-stardoc(
-    name = "docs",
-    out = "docs.md",
-    input = "helm.bzl",
-    deps = [
-        ":skylib_paths",
-        # ":bazel_tools_pkg",
-    ],
-)

Makefile

@@ -10,8 +10,8 @@ all: docs ci
 
 .PHONY: docs
 docs: deps
-	bazel build //:docs && cp bazel-bin/docs.md .
-	@chmod +w docs.md
+	bazel build //docs && cp bazel-bin/docs/docs.md docs
+	@chmod +w docs/docs.md
 
 .PHONY: deps
 ifeq ($(UNAME),Darwin)

README.md

@@ -10,7 +10,7 @@ This allows you to describe Kubernetes applications in a deterministic manner.
 
 ## Documentation
 
-* See [Rule and macro defintions](./docs.md) for macro documentation.
+* See [Rule and macro defintions](./docs/docs.md) for macro documentation.
 
 ### API
 
@@ -24,7 +24,7 @@ In your Bazel `WORKSPACE` file add this repository as a dependency:
 ```
 git_repository(
     name = "com_github_tmc_rules_helm",
-    tag = "0.1.0",
+    tag = "0.2.0",
     remote = "https://github.com/tmc/rules_helm.git",
 )
 ```

WORKSPACE

@@ -4,7 +4,7 @@ load(":repos.bzl", "helm_repositories")
 
 helm_repositories()
 
-load(":buildifier.bzl", "buildifier_repositories")
+load("//tools:buildifier.bzl", "buildifier_repositories")
 buildifier_repositories()
 
 # Start stardoc rules

docs/BUILD

@@ -0,0 +1,20 @@
+load("@io_bazel_skydoc//stardoc:stardoc.bzl", "stardoc")
+load("@bazel_skylib//:bzl_library.bzl", "bzl_library")
+
+package(default_visibility = ["//visibility:public"])
+
+load("@bazel_tools//tools/build_defs/pkg:pkg.bzl", "pkg_tar")
+
+bzl_library(
+    name = "skylib_paths",
+    srcs = ["@bazel_skylib//lib:paths.bzl"],
+)
+
+stardoc(
+    name = "docs",
+    out = "docs.md",
+    input = "//helm:helm.bzl",
+    deps = [
+        ":skylib_paths",
+    ],
+)

helm/BUILD

@@ -0,0 +1 @@
+exports_files(["helm.bzl"])

helm.bzl → helm/helm.bzl

@@ -20,8 +20,6 @@ def helm_chart(name, srcs, update_deps = False):
         srcs: Source files to include as the helm chart. Typically this will just be glob(["**"]).
         update_deps: Whether or not to run a helm dependency update prior to packaging.
     """
-    format: The format to write the Rat check report in.
-    visibility: The visibility of this rule.
     filegroup_name = name + "_filegroup"
     helm_cmd_name = name + "_package.sh"
     package_flags = ""