[Asterisk-code-review] qpp queue: new application QueueUpdate (asterisk[13])

Matthew Fredrickson asteriskteam at digium.com
Tue Nov 8 09:17:35 CST 2016


Matthew Fredrickson has posted comments on this change. ( https://gerrit.asterisk.org/4324 )

Change subject: qpp_queue: new application QueueUpdate
......................................................................


Patch Set 1: Code-Review-1

Hey Sebastien!  First off, thanks so much for submitting this to be merged back into the mainline branch.  There are a few rules, in case you weren't aware or have been previously told.  In order to merge new functionality into the 13 branch (or any other released branch), you must also submit test suite tests that cover the new functionality as well.  The master branch is the only branch that doesn't require test suite test submission for this kind of merge.

Hope all of that is understandable, and please feel free to ask any questions about the test suite on the asterisk-dev mailing list if you're not familiar with how to set it up.

-Matt

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia0ab64aed934c25b2a25022adcc7c0624224346e
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Owner: Sebastian Gutierrez <scgm11 at gmail.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Matthew Fredrickson <creslin at digium.com>
Gerrit-HasComments: No



More information about the asterisk-code-review mailing list