Skip to content

Removed const esc sequence#39

Open
andres-lowrie wants to merge 1 commit intojunyu-w:developfrom
andres-lowrie:consolidate-color-sequences
Open

Removed const esc sequence#39
andres-lowrie wants to merge 1 commit intojunyu-w:developfrom
andres-lowrie:consolidate-color-sequences

Conversation

@andres-lowrie
Copy link

Addresses: https://github.com/DrakeW/corgi/issues/34

Offloads all terminal coloring to fatih/color


How I did testing:

# From `develop`
go build
./corgi new -t exporttest
# Make a command

./corgi export exporttest
./corgi export exporttest -t shell
mv exporttest{,_beforechanges}.json
mv exporttest{,_beforechanges}

git checkout branch
go build
./corgi export exportest
./corgi export exportest -t shell

# There should be no differences
diff exporttest exporttest_beforechanges
diff exporttest.json exporttest_beforechanges.json

Loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants