bazelbuild / bazelbuild/bazel-skylib

diff_test Unified vs Context Output

Open
#354 1 comment 1 reaction 0 assignees View on GitHub
good first issue P3 type: feature request
Dominant language
Starlark
Stars
444
Forks
202
PR merge metrics
No merged PRs in 30d

Description

For the oses that use a real diff binary, can we add an option to select unified vs context diff output (`-c` vs `-u`)?

I don't mind putting in a PR, if this is something that would be accepted 😸

Contributor guide

Open the contributing guide

Research direction

Start by locating diff_test and the code paths used on operating systems that invoke a real diff binary. Trace how the current diff mode is selected, then add coverage for both unified and context output and verify the option works on the supported operating systems.

Written by the indexing model from the issue text.

Assessment

Domain
build-system
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.