bazelbuild / bazelbuild/rules_license

style: release whole repository

Open
#129 3 comments 1 reaction 0 assignees View on GitHub
Dominant language
Starlark
Stars
85
Forks
37
PR merge metrics
No merged PRs in 30d

Description

Today, developers must take care to include sources in the `filegroup(name = "standard_package")` targets which are sprinkled around the repository.
In the best case, omitting one will be caught by automated testing in the repo. However,
- it's a lot of effort to add sufficient test coverage for this
- we still won't catch everything.

I also claim there is no benefit to having Bazel build the release artifact. `git archive` is completely sufficient for this task (including pruning directories that are unnecessary in the release artifact and which make it large)

Making this change would simplify the repository so that it's more feasible for us to maintain it with a very small effort.

Contributor guide

Open the contributing guide

Research direction

Start by locating the `filegroup(name = "standard_package")` targets mentioned in the repository and reviewing how Bazel currently builds the release artifact. Compare that flow with `git archive`, including the stated directory pruning needs. Done means the release artifact no longer depends on manually maintained source lists and retains the required contents without Bazel building it.

Written by the indexing model from the issue text.

Assessment

Tech stack
git
Domain
build-system, release
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.