[PATCH] Documentation for ActiveSupport::CoreExtensions::Array::Conversions

Doc patch

+1 definitely!

+1, but why is to_sentence examples indentation different from the other examples?

Hey, I decided to do some cleanup (and add some more info) when I was reviewing this. Changes:

• Changed method descriptions to say "returns ..." instead of "converts" since they're not bang methods
• Fixed indentation re: tarmo's suggestion
• Wrapped docs to 80 chars where possible
• Added more info and another example about the other to_xml options: :dasherize, :skip_instruct, :skip_types, and the ability to call with a block; cleaned up and rewrote descriptions for other options
• Changed your beets example a bit since, I think, carrots aren't beets? First time I've looked up vegetables on Wikipedia while working on a Rails patch, that's for sure.
• Other miscellaneous monospacing and wording
• Updated for edge.

Hope this helps.

These look good but I'm afraid the patch doesn't apply cleanly anymore. I'll be glad to apply an updated patch.

OK, cool. Let me upload a clean patch...

added documentation for to_formatted_s. I had another ticket open for this addition, as well to_xml. Figured it is best to combine these efforts by adding to_formatted_s to your work done here.

+1

Don't +1 and verify your own patch :)

Seriously, though, I need to fix some consistency problems with the new to_formatted_s addition. I'll do that in a sec and put up a new patch. Then, let's get one of the previous verifiers (or someone new) to do another verification.

I've been meaning to do this for a week now and it got back-burnered. Sorry about that.

hehe, sorry about that. :)

OK, here we go. I rewrote the to_formatted_s docs to be more consistent with the other methods. I think it's a little more clear and concise, too. And, no more spelling or grammar errors ("behavious" etc.).

Also applies cleanly again.

Let's get another verifier in here to test this out, and then we can go forward.