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

New screenshot #115

Closed
bmitch opened this issue Oct 3, 2017 · 3 comments
Closed

New screenshot #115

bmitch opened this issue Oct 3, 2017 · 3 comments

Comments

@bmitch
Copy link
Owner

bmitch commented Oct 3, 2017

Still not entirely happy with the screenshot in the readme.md

  • It has a command prompt at the top and the bottom. Could remove both, neither are really needed.
  • It contains a really long file name (the CyclomaticComplexityAssessor.php) in the results which forces the table to be very wide. If that file was ignored the table would be smaller in width which I think would improve the screenshot.
@vrishank97
Copy link
Contributor

Can I take this up?

@bmitch
Copy link
Owner Author

bmitch commented Oct 3, 2017

@vrishank97 Yeah sure, thanks.

@bmitch
Copy link
Owner Author

bmitch commented Oct 6, 2017

@vrishank97 if you are still interested in this, I just updated the master branch with the new scoring algorithm. So if you can use that branch it would be best. Thanks!

@bmitch bmitch closed this as completed Oct 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants