bazel-contrib / bazel-contrib/rules_oci

Rename oci_load to oci_daemon_load for clarity

Open
#785 0 comments 2 reactions 0 assignees View on GitHub
Dominant language
Starlark
Stars
423
Forks
213
Avg merge
53m
Merged PRs (30d)
1

Description

We'dlie like to rename oci_load to oci_daemon_load for better clarity in 3.x version of rules_oci.

Currently `oci_load` confuses people into thinking it is orthogonal to oci.pull but fetching from the daemon/tarball which is completely wrong. I'd like to rename it be more aligned with what it does.

Contributor guide

Open the contributing guide

Research direction

Search the repository for `oci_load` and inspect its rule definition and every reference. Rename the public entry point to `oci_daemon_load`, update affected references, and verify that no unintended `oci_load` references remain.

Written by the indexing model from the issue text.

Assessment

Domain
build-system
Issue type
Refactor
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.