[development] patches that need reviews/work
Gabor Hojtsy
gabor at hojtsy.hu
Fri Jun 1 11:43:50 UTC 2007
Ashraf Amayreh wrote:
> Thanks to Dries' wise decision, we can now really put some extra efforts
> to make this a perfect release, there are patches that need reviews,
> others that need some work to minimize their performance impact, and
> others that are RTBC but are still not.
>
> To ease this and make it more effective, everyone who has a patch that
> needs reviews, some work or is RTBC but is still not, please post it in
> this thread to get it under lights. Please don't post another
> developer's patches unless you're in sync with him so that this won't
> get filled with repetitive posts. A short description of each patch and
> what it does should be helpful as long as it's no longer than a
> paragraph, this is a second reason the patch author is in a better
> position to post here rather than someone else, interested parties can
> see the details in the issue page. By doing this, we can centralize all
> our efforts on the patches in this thread.
>
> Finally, let's avoid giving opinions on these patches over here to
> streamline this thread and attempt to make it as direct as possible. The
> issue page can be used for that.
>
> My patch is a simple one that Moshe has agreed is RTBC, so I hope we can
> get it in and turn to other important patches.
>
> http://drupal.org/node/114938
Some advice:
- next time try to follow your own suggestion and explain what your
patch does when trying to shed some light on it!
- for the same reason, having such patches in a generic titled thread
can quickly go out of focus as far as I see...
Gabor
More information about the development
mailing list