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

Add documentation for --use-colour #1263

Merged
merged 1 commit into from
May 6, 2018
Merged

Add documentation for --use-colour #1263

merged 1 commit into from
May 6, 2018

Conversation

palotasb
Copy link
Contributor

The documentation for --use-colour option was missing.

I added it based on the -? output a Catch test program gave me and the comments on #950

Description

GitHub Issues

@horenmar
Copy link
Member

Huh, so it was. The linked issue does not seem correct though, did you mean #590 ?

@codecov
Copy link

codecov bot commented Apr 26, 2018

Codecov Report

Merging #1263 into master will increase coverage by 0.83%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1263      +/-   ##
==========================================
+ Coverage   81.43%   82.27%   +0.83%     
==========================================
  Files         126      124       -2     
  Lines        4810     4761      -49     
==========================================
  Hits         3917     3917              
+ Misses        893      844      -49

@palotasb
Copy link
Contributor Author

palotasb commented Apr 26, 2018 via email

@horenmar
Copy link
Member

The link in the commit message should go to the correct issue before it is merged to master.

Thanks for the PR btw.

The documentation added is based on output from `-?` (help),
and comments to #590.
@horenmar horenmar merged commit dd99a66 into catchorg:master May 6, 2018
@horenmar
Copy link
Member

horenmar commented May 6, 2018

I had some spare time so I made the changes myself.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants