On Sun, 19 Jan 2003, Rich Bowen wrote:
> On Sun, 19 Jan 2003, Dave Rolsky wrote:
>
> > On Sun, 19 Jan 2003, Rich Bowen wrote:
> >
> > > I presume this is actually intentional:
> > >
> > > t/03components....ok 19/44Can't locate object method "ical" via package
> > > "DateTime" at t/03components.t line 36.
> > >
> > > If so, could you please remove the tests that require this method, so
> > > that we can have a passing test suite.
> > >
> > > Along that same line:
> > >
> > > Warning: the following files are missing in your kit:
> > > t/05ical.t
> > >
> > > That test was removed intentionally? If so, please remove from MANIFEST
> > > also.
> >
> > Oops, yep. It was all intentional. The contents of t/05ical.t will be
> > used for DateTime::Formats::ICal though. I will check that in soon, but
> > I'm still trying to decide between "Format" and "Formats", since I
> > realized they _both_ have trouble when interpreted as a verb, so I'm now
> > leaning back towards Format because nothing else is likely to be plural.
>
> +1 for singular. I don't like plural class names.
We already had this discussion! Formats was the best offering. Any change
needs to be an improvement, imo.
- nick
~~~~~~~~~~~~~~~~~~~~
Nick Tonkin {|8^)>
