Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

This project is archived and is in readonly mode.

Documentation for Array#sample

#4995

Hi guys,

This patch introduce documentation and some examples for the Array#sample method.

Reported by Thiago Pradi · June 28th, 2010 @ 03:34 PM

State: resolved
Milestone: none
Assigned to: Xavier Noria Xavier Noria
Importance: Low

Activity

  1. Xavier Noria
    Xavier Noria
    • Importance changed from to Low

    Hi Thiago, we normally do not document backports because they are known methods, or methods with existing documentation anyway. We do document small differences if there are any, because a backport is assumed otherwise to behave exactly the same in all edge cases, invalid arguments, etc.

    That being said that's not a hard rule and the patch is welcome, could you please commit it yourself via docrails? I'll merge it into master later. docrails has public write access, just push the changes.

    Thanks!

    June 28th, 2010 @ 04:03 PM

  2. Thiago Pradi
    Thiago Pradi

    Hi Xavier,

    I'll commit to docrails.

    Thanks for the reply and for the clarification!

    June 28th, 2010 @ 05:21 PM

  3. Rohit Arondekar
    Rohit Arondekar
    • State changed from new to resolved

    Assuming this commit in docrails to resolve this ticket http://github.com/lifo/docrails/commit/0175c470c01c22db2f54c3912958...

    June 29th, 2010 @ 10:45 AM