-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
PR: Update readme and organize badges #3336
Conversation
goanpeca
commented
Jul 30, 2016
•
edited
Loading
edited
- Reorganize badges for build and project status
- Include Quantified Code badge
- Include Gitter chat bagde
My replies:
|
What happened here? Why did you close it? |
Frustration... On your comments:
It does, it conveys that the codebase is following some quality standards and makes issues in the codebase visible. The idea is to be transparent, as with the covergae despite it being low we give visibility to it and that in turns will steer the project to increasing coverage. Same with quantified code.
Ok
Makes sense
I will but please enable circle ci, it is kinda if a bummer that you just shut it down. I can have it working real quick (I have enable circle ci on several projects now... anaconda-client, conda-manager, navigator...) |
Ok, then it's fine to have it :-p
I don't have time to deal with another CI service right now. Besides, after a long time, Travis and AppVeyor are working really well now, so I don't want to touch them. Sorry, I know it's a bummer, but let's just focus on getting 3.0 out of the door for now. Please do my couple of suggested changes to merge this one :-) |
I'll merge this one and do my suggested changes in master :-) |