You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
codeowners_register() should take a filepath parameter that defaults to .github/CODEOWNERS (or another one of the valid GitHub CODEOWNERS paths). This will be consumed at build time.
Building
In an ideal world, we'd be able to bazel build ..., and the codeowners() rules would be picked up and compiled into a file that's output to filepath. Unfortunately, we can't do that. What we can do is provide a target that is runnable and does the same work; we'd only need to provide the name -- this would be similar to other rules such as bazelbuild/bazel-gazelle and bazelbuild/buildtools, and would be invoked as such:
Output Path
codeowners_register()
should take afilepath
parameter that defaults to.github/CODEOWNERS
(or another one of the valid GitHub CODEOWNERS paths). This will be consumed at build time.Building
In an ideal world, we'd be able to
bazel build ...
, and thecodeowners()
rules would be picked up and compiled into a file that's output tofilepath
. Unfortunately, we can't do that. What we can do is provide a target that is runnable and does the same work; we'd only need to provide thename
-- this would be similar to other rules such asbazelbuild/bazel-gazelle
andbazelbuild/buildtools
, and would be invoked as such:This would parse all of the
codeowners()
rules and create the generated file at the specified output path (fromcodeowners_register()
, above).I think following the pattern of established tool integrations like
bazel-gazelle
andbuildifier
(viabuildtools
) is a great way forward.Depends-On: #25
Sourced from: #13 (comment)
The text was updated successfully, but these errors were encountered: