Reviewing merge request #39: Added debugging information
commit e771051a2a8173c019dcad29b2eab2c7fde2d9bb
Author: Andres Gomez <agomez@igalia.com>
Date: Thu Feb 2 01:09:27 2012 +0200
Added debugging information
DEBUG_INFO | 122 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
1 files changed, 122 insertions(+), 0 deletions(-)
- Requested by:
Andres Gomez - Status: Closed
- Source repository: agomez-quillmetadata:DEBUG_INFO_added
- Target repository: quillmetadata:master
- Created at: February 3rd, 2012 18:09
Commits that would be merged:
Comments
Pushed new version 1
+ For running the unit tests, only thing to be done is invoking each
“only thing” or “the only thing”?
Juan A. Suárez Romero |
2012-02-08 12:31:29 UTC
on DEBUG_INFO
+ Actually, that way we will also know how good are our unit tests and + how far are they covering our code.
I'd say “the unit tests” and “covering the code”
Juan A. Suárez Romero |
2012-02-08 12:32:10 UTC
on DEBUG_INFO
+ -- Andres Gomez <agomez@igalia.com> Tue, 31 Jan 2012 19:32:00 +0100
Is expected to add this line in the file?
Juan A. Suárez Romero |
2012-02-08 12:32:56 UTC
on DEBUG_INFO
Looks fine to me, simply check Juan’s comments. Merge it!
Add a new comment:
Login or create an account to post a comment