Project

General

Profile

Bug #8821

Add documentation for Time#w3cdtf

Added by Steve Klabnik about 3 years ago. Updated almost 3 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Zachary Scott
ruby -v:
2.1.0-dev
Backport:
1.9.3: UNKNOWN, 2.0.0: UNKNOWN
[ruby-core:56814]

Description

These monkey-patches on Time weren't documented inside of RSS. Now they are.

0001-Add-documentation-for-the-w3cdtf-methods-on-Time.patch View (1.37 KB) Steve Klabnik, 08/27/2013 06:15 AM

Associated revisions

Revision 43600
Added by Zachary Scott almost 3 years ago

  • lib/rss/rss.rb: [DOC] document Time#w3cdtf by @steveklabnik [Bug #8821] ありがとうかににく

Revision 43600
Added by Zachary Scott almost 3 years ago

  • lib/rss/rss.rb: [DOC] document Time#w3cdtf by @steveklabnik [Bug #8821] ありがとうかににく

Revision 43600
Added by Zachary Scott almost 3 years ago

  • lib/rss/rss.rb: [DOC] document Time#w3cdtf by @steveklabnik [Bug #8821] ありがとうかににく

History

#1 [ruby-core:57397] Updated by Zachary Scott about 3 years ago

I would just say "this method", instead of "the method".

Otherwise, its good. Thank you!

#2 [ruby-core:57766] Updated by Steve Klabnik almost 3 years ago

Great! Do you need me to modify the patch, or is this simple enough to change as you apply it?

#3 [ruby-core:57881] Updated by Zachary Scott almost 3 years ago

I will modify it when I get around to committing the patch (unless you submit one before then), thank you!

#4 [ruby-core:57882] Updated by Zachary Scott almost 3 years ago

  • Status changed from Open to Assigned

#5 Updated by Zachary Scott almost 3 years ago

  • Status changed from Assigned to Closed
  • % Done changed from 0 to 100

This issue was solved with changeset r43600.
Steve, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.

  • lib/rss/rss.rb: [DOC] document Time#w3cdtf by @steveklabnik [Bug #8821] ありがとうかににく

Also available in: Atom PDF