5981796586
57%
master: 50%

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.2%) from 57.267%

Build # 5981796586

Build Type

push

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()
```

Run Details

524 of 918 relevant lines covered (57.08%)

8.85 hits per line

Jobs

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

Source Files on build 5981796586

Detailed source file information is not available for this build.

muesli / termenv / 5981796586
57%
master: 50%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 5981796586

muesli / termenv / 5981796586 57% master: 50%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 5981796586

muesli / termenv / 5981796586
57%
master: 50%

README BADGES
x