You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@activemq.apache.org by dudaerich <gi...@git.apache.org> on 2016/01/27 14:38:00 UTC

[GitHub] activemq-artemis pull request: ARTEMIS-365 - [Artemis Testsuite] A...

GitHub user dudaerich opened a pull request:

    https://github.com/apache/activemq-artemis/pull/355

    ARTEMIS-365 - [Artemis Testsuite] AddressControlTest#testGetNumberOfP…

    …ages fails

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/dudaerich/activemq-artemis ARTEMIS-365

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/activemq-artemis/pull/355.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #355
    
----
commit 442410372e07604af5a597b1705915ddd430b450
Author: Erich Duda <ed...@redhat.com>
Date:   2016-01-27T13:37:21Z

    ARTEMIS-365 - [Artemis Testsuite] AddressControlTest#testGetNumberOfPages fails

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] activemq-artemis pull request: ARTEMIS-365 - [Artemis Testsuite] A...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

    https://github.com/apache/activemq-artemis/pull/355


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] activemq-artemis pull request: ARTEMIS-365 - [Artemis Testsuite] A...

Posted by dudaerich <gi...@git.apache.org>.
Github user dudaerich commented on the pull request:

    https://github.com/apache/activemq-artemis/pull/355#issuecomment-175693517
  
    @clebertsuconic, you're welcome :) Yes, I've tested it with master and it's ok now.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] activemq-artemis pull request: ARTEMIS-365 - [Artemis Testsuite] A...

Posted by clebertsuconic <gi...@git.apache.org>.
Github user clebertsuconic commented on the pull request:

    https://github.com/apache/activemq-artemis/pull/355#issuecomment-175661099
  
    Awesome!!! I was working on those :)
    
    Did you test it against master and it's passing now?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] activemq-artemis pull request: ARTEMIS-365 - [Artemis Testsuite] A...

Posted by clebertsuconic <gi...@git.apache.org>.
Github user clebertsuconic commented on the pull request:

    https://github.com/apache/activemq-artemis/pull/355#issuecomment-175661339
  
    @dudaerich  Thank you so much... you saved my day with these fixes :)


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---