You can remove compatibility check (including the one that's there). IronRuby only supports MRI 1.9.
Tomas -----Original Message----- From: ironruby-core-boun...@rubyforge.org [mailto:ironruby-core-boun...@rubyforge.org] On Behalf Of Enrico Sada Sent: Tuesday, August 02, 2011 9:45 AM To: ironruby-core@rubyforge.org Subject: [Ironruby-core] Code Review - Time#strftime ignore invalid directives on format string i asked a pull request ( https://github.com/IronLanguages/main/pull/28 ) for fix Time#strftime behaviour on invalid directives on format string, ex: ruby 1.8: Time.now.strftime '%$' => '$' ruby 1.9: Time.now.strftime '%$' => '%$' this make green a mspec test on core/time/strftime_spec.rb I have some question: 1) i added a check for ruby compatibility >= ruby 1.9, is needed? 2) i removed 'fails:' from ironruby-tags-19/core/time/strftime_tags.txt is correct? 3) usually code review is in mailing list or github pull request? (so i dont need to write two times the same questions) _______________________________________________ Ironruby-core mailing list Ironruby-core@rubyforge.org http://rubyforge.org/mailman/listinfo/ironruby-core _______________________________________________ Ironruby-core mailing list Ironruby-core@rubyforge.org http://rubyforge.org/mailman/listinfo/ironruby-core