tinted-theming / tinted-shell

Base16 and Base24 for Shells
https://github.com/tinted-theming/home
MIT License
65 stars 217 forks source link

[Bug report] Git delta hook unknown option and light value issues #32

Closed JamyGolden closed 1 year ago

JamyGolden commented 1 year ago

Describe the bug

@bezhermoso I've noticed 2 issues with the new git delta hook:

Screenshots

These screenshots are with the removal of the -g option for bc command on line 19.

Works correctly when setting a dark theme image

Does not work correctly when setting a light theme (notice # is_light_theme = 01 on line 18) image

System

Operating system: Pop!Os (Linux)

Terminal: Alacritty

JamyGolden commented 1 year ago

I've created a PR to fix this, @bezhermoso maybe you can do a quick review? https://github.com/tinted-theming/base16-shell/pull/33

JamyGolden commented 1 year ago

This issue has been fixed. Look at the merged PR for more details https://github.com/tinted-theming/base16-shell/pull/33