5981796828
57%
master: 58%

Ran 26 Aug 2023 12:52AM UTC

Jobs 1

Files 12

Run time 2s

Badge

Embed ▾

Committed 26 Aug 2023 12:51AM UTC coverage: 57.081% (-0.6%) from 57.655%

Build # 5981796828

Build Type

Pull #149

github

Committed by

aymanbagabas

Commit Message

fix: read fore/background colors on first access

Caching the colors in OutputOption makes the order of options important.
For example, if you're using WithUnsafe _after_ WithColorCache, the
colors wouldn't respect the `unsafe` option since the colors get cached
_during_ the execution of WithColorCache option.

Another example, is the use of a custom environ using the WithEnvironment
option. If WithEnvironment comes _after_ WithColorCache, reading the
terminal colors won't respect the values in the environment.

Instead, don't read the colors in WithColorCache and make the user
opt-in if they want to cache the colors right after creating a new
output using ForegroundColor and BackgroundColor.

```go
// To cache the colors after creating an output
output := NewOutput(os.Stderr)
_ = output.ForegroundColor()
_ = output.BackgroundColor()
```

Pull Request Pull Request #149: fix: read fore/background colors on first access

Run Details

524 of 918 relevant lines covered (57.08%)

8.85 hits per line

Uncovered Existing Lines

Lines	Coverage	∆	File
10	70.1	-4.9%	output.go

Jobs

ID	Job ID	Ran	Files	Coverage
1	5981796828.1	26 Aug 2023 12:52AM UTC	12	57.08	GitHub Action Run

Source Files on build 5981796828

Coverage	∆	File	Lines	Relevant	Covered	Missed	Hits/Line

muesli / termenv / 5981796828
57%
master: 58%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Uncovered Existing Lines

Jobs

Source Files on build 5981796828

muesli / termenv / 5981796828 57% master: 58%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Uncovered Existing Lines

Jobs

Source Files on build 5981796828

muesli / termenv / 5981796828
57%
master: 58%

README BADGES
x