How to handle small ancillary bug fixes?

View: New views
2 Messages — Rating Filter:   Alert me  

How to handle small ancillary bug fixes?

by Rory Plaire :: Rate this Message:

| View Threaded | Show Only this Message

I'm debugging an issue in `Castle.MicroKernel.Handlers.ParentHandlerWithChildResolver` and notice that `TryResolve` calls `parentHandler.Resolve` instead of `parentHandler.TryResolve`.
 
As I'm going to be submitting an issue with a proposed fix for `Castle.MicroKernel.Handlers.ParentHandlerWithChildResolver`, should I just put this in the same changeset, or are separate bug reports / tests preferred?
 
-r

--
You received this message because you are subscribed to the Google Groups "Castle Project Development List" group.
To post to this group, send email to castle-project-devel@....
To unsubscribe from this group, send email to castle-project-devel+unsubscribe@....
For more options, visit this group at http://groups.google.com/group/castle-project-devel?hl=en.

Re: How to handle small ancillary bug fixes?

by Krzysztof Kozmic-2 :: Rate this Message:

| View Threaded | Show Only this Message

Hi Rory,

It's fine either way. Submitting tests and change that fixes them in one
commit is fine.

cheers,
Krzysztof

On 19/01/2012 10:40 AM, Rory Plaire wrote:

> I'm debugging an issue in
> `Castle.MicroKernel.Handlers.ParentHandlerWithChildResolver` and
> notice that `TryResolve` calls `parentHandler.Resolve` instead of
> `parentHandler.TryResolve`.
> As I'm going to be submitting an issue with a proposed fix for
> `Castle.MicroKernel.Handlers.ParentHandlerWithChildResolver`, should I
> just put this in the same changeset, or are separate bug reports /
> tests preferred?
> -r
> --
> You received this message because you are subscribed to the Google
> Groups "Castle Project Development List" group.
> To post to this group, send email to
> castle-project-devel@....
> To unsubscribe from this group, send email to
> castle-project-devel+unsubscribe@....
> For more options, visit this group at
> http://groups.google.com/group/castle-project-devel?hl=en.

--
You received this message because you are subscribed to the Google Groups "Castle Project Development List" group.
To post to this group, send email to castle-project-devel@....
To unsubscribe from this group, send email to castle-project-devel+unsubscribe@....
For more options, visit this group at http://groups.google.com/group/castle-project-devel?hl=en.