[Let''s give this a go, shall we?]
As per http://wiki.rubyonrails.org/rails/pages/PatchRequirements, I''m
looking for reviewers and approvals for the following patches:
* 8714 -- Fix the actionpack test suite against a problem that can happen
when the test load order gets mucked up.
* 8961 -- Fix the ssl_requirement test suite to work with the current Rails
tree.
* 8962 -- Fix the ssl_requirement plugin to allow the overloading of
ssl_allowed?
* 8997 -- Improve the ssl_requirement README a bit.
- Matt
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"Ruby on Rails: Core" group.
To post to this group, send email to rubyonrails-core@googlegroups.com
To unsubscribe from this group, send email to
rubyonrails-core-unsubscribe@googlegroups.com
For more options, visit this group at
http://groups.google.com/group/rubyonrails-core?hl=en
-~----------~----~----~----~------~----~------~--~---
Could we get a Trac for report unreviewed tickets to make this process easier? Maybe a keyword or something like that? --Jeremy On 7/16/07, Matthew Palmer <mpalmer@hezmatt.org> wrote:> > [Let''s give this a go, shall we?] > > As per http://wiki.rubyonrails.org/rails/pages/PatchRequirements, I''m > looking for reviewers and approvals for the following patches: > > * 8714 -- Fix the actionpack test suite against a problem that can happen > when the test load order gets mucked up. > * 8961 -- Fix the ssl_requirement test suite to work with the current Rails > tree. > * 8962 -- Fix the ssl_requirement plugin to allow the overloading of > ssl_allowed? > * 8997 -- Improve the ssl_requirement README a bit. > > - Matt > > > >-- http://www.jeremymcanally.com/ My free Ruby e-book: http://www.humblelittlerubybook.com/book/ My blogs: http://www.mrneighborly.com/ http://www.rubyinpractice.com/ --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Ruby on Rails: Core" group. To post to this group, send email to rubyonrails-core@googlegroups.com To unsubscribe from this group, send email to rubyonrails-core-unsubscribe@googlegroups.com For more options, visit this group at http://groups.google.com/group/rubyonrails-core?hl=en -~----------~----~----~----~------~----~------~--~---
On Mon, Jul 16, 2007 at 04:33:16PM -0400, Jeremy McAnally wrote:> Could we get a Trac for report unreviewed tickets to make this process > easier? Maybe a keyword or something like that?If it were up to me, I''d make it a custom field, "Patch status". Choices would include "Needs a patch", "Needs tests", "Needs docs", "Needs review", and "Approved". Considering that every ticket should fall into one of those categories, using keywords seems a little too free-form and prone to error. http://trac.edgewall.org/wiki/TracTicketsCustomFields if the Trac admins aren''t familiar with custom fields. - Matt -- "Mr... Baggins. It seems you have been leading... two lives. In one... you are an ordinary hobbit... you organize your uncle''s diary... give generous gifts to children. In the other... you are the Ring Bearer... charged to carry the One Ring to Mount Doom. One life has... a future, Mr... Baggins, and the other..." --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Ruby on Rails: Core" group. To post to this group, send email to rubyonrails-core@googlegroups.com To unsubscribe from this group, send email to rubyonrails-core-unsubscribe@googlegroups.com For more options, visit this group at http://groups.google.com/group/rubyonrails-core?hl=en -~----------~----~----~----~------~----~------~--~---
As http://dev.rubyonrails.org/report/12 is totally empty at this point, all the open patches needs to be reviewed :) -Pratik On 7/16/07, Jeremy McAnally <jeremymcanally@gmail.com> wrote:> > Could we get a Trac for report unreviewed tickets to make this process > easier? Maybe a keyword or something like that? > > --Jeremy > > On 7/16/07, Matthew Palmer <mpalmer@hezmatt.org> wrote: > > > > [Let''s give this a go, shall we?] > > > > As per http://wiki.rubyonrails.org/rails/pages/PatchRequirements, I''m > > looking for reviewers and approvals for the following patches: > > > > * 8714 -- Fix the actionpack test suite against a problem that can happen > > when the test load order gets mucked up. > > * 8961 -- Fix the ssl_requirement test suite to work with the current Rails > > tree. > > * 8962 -- Fix the ssl_requirement plugin to allow the overloading of > > ssl_allowed? > > * 8997 -- Improve the ssl_requirement README a bit. > > > > - Matt > > > > > > > > > > -- > http://www.jeremymcanally.com/ > > My free Ruby e-book: > http://www.humblelittlerubybook.com/book/ > > My blogs: > http://www.mrneighborly.com/ > http://www.rubyinpractice.com/ > > > >-- http://m.onkey.org --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Ruby on Rails: Core" group. To post to this group, send email to rubyonrails-core@googlegroups.com To unsubscribe from this group, send email to rubyonrails-core-unsubscribe@googlegroups.com For more options, visit this group at http://groups.google.com/group/rubyonrails-core?hl=en -~----------~----~----~----~------~----~------~--~---