bazelbuild / bazelbuild/bazel

Merge SolibSymlinkAction with SymlinkAction

Open
#7,700 2 comments 0 reactions 0 assignees View on GitHub
not stale P3 team-Remote-Exec type: process
Dominant language
Java
Stars
25.8k
Forks
4.6k
Avg merge
2d 20h
Merged PRs (30d)
72

Description

`SolibSymlinkAction` should be turned into a factory that creates a `SymlinkAction` as their `execute` method is effectively identical. The benefit of this is that we'll have one fewer native action implementation.

Contributor guide

Open the contributing guide

Research direction

Locate SolibSymlinkAction and SymlinkAction, then compare their execute methods and construction paths. Confirm that SolibSymlinkAction becomes a factory producing SymlinkAction and that the native action implementation count is reduced without changing execution behavior.

Written by the indexing model from the issue text.

Assessment

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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.