Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: fixed ANSI escape code error on local testing #202

Merged
merged 3 commits into from
Nov 13, 2020

Conversation

gautamkrishnar
Copy link
Contributor

@gautamkrishnar gautamkrishnar commented Nov 13, 2020

Fixed ANSI escape code error on local testing. When you test the code locally it was failing. In CI it was returning success.

Error:
Screenshot 2020-11-13 at 8 11 47 PM

Reason for the error:
Screenshot 2020-11-13 at 8 46 09 PM

What kind of change does this PR introduce?
Disable chalk's color using FORCE_COLOR environment variable

Did you add tests for your changes?
n/a

If relevant, did you update the documentation?
n/a

Summary
n/a

Does this PR introduce a breaking change?
no

Other information
m/a

@welcome
Copy link

welcome bot commented Nov 13, 2020

Thanks for opening this pull request! Please check out our contributing guidelines.

@gautamkrishnar gautamkrishnar marked this pull request as draft November 13, 2020 15:41
@gautamkrishnar gautamkrishnar marked this pull request as ready for review November 13, 2020 15:59
@jamesgeorge007 jamesgeorge007 added this to In progress in Roadmap via automation Nov 13, 2020
@jamesgeorge007 jamesgeorge007 merged commit 6b4a567 into madlabsinc:develop Nov 13, 2020
Roadmap automation moved this from In progress to Done Nov 13, 2020
@welcome
Copy link

welcome bot commented Nov 13, 2020

congrats
Congrats on your very first contribution 👏 Looking forward to having more from your side

@gautamkrishnar gautamkrishnar deleted the fix-local-color branch November 13, 2020 16:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Roadmap
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants