This project is archived and is in readonly mode.

#4995 ✓resolved
Thiago Pradi

Documentation for Array#sample

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

Hi guys,

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

Comments and changes to this ticket

  • Xavier Noria

    Xavier Noria June 28th, 2010 @ 04:03 PM

    • 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!

  • Thiago Pradi

    Thiago Pradi June 28th, 2010 @ 05:21 PM

    Hi Xavier,

    I'll commit to docrails.

    Thanks for the reply and for the clarification!

  • Rohit Arondekar

    Rohit Arondekar June 29th, 2010 @ 10:45 AM

    • State changed from “new” to “resolved”

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

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile »

<h2 style="font-size: 14px">Tickets have moved to Github</h2>

The new ticket tracker is available at <a href="https://github.com/rails/rails/issues">https://github.com/rails/rails/issues</a>

Attachments

Tags

Pages