keith / buildifier-prebuilt

A bazel toolchain for using prebuilt binaries for buildifier and buildozer
MIT License
35 stars 13 forks source link

Add the ability to specify the version and tool from `buildtools`. #7

Closed cgrindel closed 2 years ago

cgrindel commented 2 years ago

Related to #5.

cgrindel commented 2 years ago

@keith I am not sure if you have had a chance to review this PR, yet. I have a follow-on PR that adds a utility to generate a buildtools_assets declaration for the latest buildtools release or, if a release tag is specified, it will produce it for the specified release.

cgrindel commented 2 years ago

The asset declaration generation PR is ready. If you would like to see it while reviewing this PR, here is a link.

cgrindel commented 2 years ago

@keith I made some updates based upon your comments. I added a new issue (#9) to track an issue with buildozer usage. Let me know your thoughts.