I wish I could say something more enlightening, but I would say that
if it can be demonstrated that the existing behavior is wrong and that
the new behavior is correct across all existing tests, there is a
significant likelihood that it would be considered for inclusion. The
patch should include unit tests and would need to be reviewed. We
can't guarantee at any point if any particular effort will be
utilized, but we do welcome the help in improving NHibernate. If you
put the work in and get it to the point of submitting a pull request,
we'll certainly take a look at it. Check out "Contributor Guide.html"
in the root of the repo for related details that should help you get
started.
Patrick Earl
On Tue, Sep 27, 2011 at 7:44 PM, Isaac Cambron <[email protected]> wrote:
> I'd very much like to patch NH-2900, but since it'll only be my second
> patch, I figured I'd ask about before I dive in:
> 1. Since I just filed it, does it make sense?
> 2. If I patched this, would it be accepted?
> 3. Any pointers one where I should start? I know my way around the code base
> a little bit, but if anyone has some advice on how to attack this issue, I'd
> appreciate it.
> Thanks,
> Isaac
