bazelbuild / stardoc

Stardoc: Starlark Documentation Generator
Apache License 2.0
108 stars 45 forks source link

Add bzl_library for html_tables_stardoc.bzl #104

Closed cvrebert closed 3 years ago

cvrebert commented 3 years ago

Previously, there wasn't any bzl_library which included it. Which prevents further downstream bzl_librarys from being defined with fully accurate deps.

brandjon commented 3 years ago

Any opinion on including that .bzl in the existing stardoc_lib target?

+@tetromino

cvrebert commented 3 years ago

No strong opinion. One bzl_library per public .bzl file is a common pattern and minimizes deps trees, but I'm not on Bazel Team.