[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