[infinispan-dev] Unit test PR for master

Sanne Grinovero sanne at infinispan.org
Thu Oct 25 11:37:02 EDT 2012


Hi Vladimir,
I'm not sure I understood you. are these pulls you mention having failing tests?
I've sent a test related PR yesterday which was just changing a single
test from using deprecated code to new code, there is nothing to fix.

Sanne


On 25 October 2012 15:52, Vladimir Blagojevic <vblagoje at redhat.com> wrote:
> Hi,
>
> I noticed that we recently have had many PR for unit tests integration
> into master. Would it not make more sense to integrate unit tests with
> actual fixes rather than having them merged directly to master as PRs?
> I'd say leave a unit test attached/referenced in JIRA and a
> developer/contributor fixing the issue will integrate both the fix and
> the unit test in a single PR.
>
> WDYT?
>
> Regards,
> Vladimir
>
>
>
> _______________________________________________
> infinispan-dev mailing list
> infinispan-dev at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/infinispan-dev


More information about the infinispan-dev mailing list