[Asterisk-code-review] Add test for PJSIP forward loops. (testsuite[master])

Joshua Colp asteriskteam at digium.com
Mon Apr 20 15:51:34 CDT 2015


Joshua Colp has posted comments on this change.

Change subject: Add test for PJSIP forward loops.
......................................................................


Patch Set 5:

(1 comment)

https://gerrit.asterisk.org/#/c/119/5//COMMIT_MSG
Commit Message:

Line 17: Address review feedback
       : 
       : * Remove trailing whitespace from test-config.yaml
       : * Change tag in test-config.yaml to "pjsip"
       : 
       : Change-Id: I5675c6d1642c53c7fcd8e0d5847b0c3d66798197
> Okay, so that brings up a good point.
The commit message here is the one that is going into the tree. Personally I just discard my subsequent commit messages because they don't add value to what is going into the tree. If things have changed in a fashion that needs to be mentioned in the commit message, I already the main one.


-- 
To view, visit https://gerrit.asterisk.org/119
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I5675c6d1642c53c7fcd8e0d5847b0c3d66798197
Gerrit-PatchSet: 5
Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-Owner: Mark Michelson <mmichelson at digium.com>
Gerrit-Reviewer: Jared K. Smith <jaredsmith at jaredsmith.net>
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Mark Michelson <mmichelson at digium.com>
Gerrit-Reviewer: Matt Jordan <mjordan at digium.com>
Gerrit-HasComments: Yes



More information about the asterisk-code-review mailing list