bazelbuild / bazelbuild/rules_license

Create py_binary target for add_licenses.py

Open Beginner friendly
#22 0 comments 0 reactions 0 assignees View on GitHub
P3
Dominant language
Starlark
Stars
85
Forks
37
PR merge metrics
No merged PRs in 30d

Description

Currently, the `add_licenses.py` tool must be run with:
```
LC_ALL="en_US.UTF-8" admin/refresh_spdx/add_licenses.py
```
Instead, declare a `py_binary` target in `admin/refresh_spdx/BUILD` so the `add_licenses.py` tool can be run with:
```
bazel run //admin/refresh_spdx:add_licenses
```

See PR review comment: https://github.com/bazelbuild/rules_license/pull/17#discussion_r789010877

Contributor guide

Open the contributing guide

Research direction

Open admin/refresh_spdx/BUILD and inspect the existing add_licenses.py tool in admin/refresh_spdx. Add the requested py_binary target, then run bazel run //admin/refresh_spdx:add_licenses to verify that the tool is runnable through Bazel.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
build-system
Issue type
Feature
Difficulty
1/5
Estimated time
Under an hour
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
65/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.