develooper Front page | perl.perl5.porters | Postings from November 2012

[perl #115838] t/op: Move tests not using t/ into a separate directory

Thread Previous | Thread Next
James E Keenan via RT
November 25, 2012 18:19
[perl #115838] t/op: Move tests not using t/ into a separate directory
Message ID:
On Sat Nov 24 18:11:10 2012, wrote:
> On 11/24/2012 05:57 PM, Ricardo Signes wrote:
> > * Karl Williamson <> [2012-11-20T22:54:23]
> >> It would be good to have a comment in each file that indicates what
> >> the features being tested are that prohibit its use of
> >

Earlier in this ticket I provided a list of 13 t/op/*.t files which
currently do *not* use t/  If we could get some eyeballs on
these files so that we can identify the prohibitory factor, we can make
some progress.

I am willing to create a branch to manage this transition -- but let's
first clean up the loose ends from the St Louis Perl Hackathon out of
which this discussion of t/ emerged.  These tickets, which add
descriptions to tests lacking them, need code review:

> > Absolutely.  I would really like to have the "can't use"
> tests isolated
> > and commented to help future contributors.  I've been bitten by the
> same "you
> > actually can't do that" feedback in the past, and it was quite a
> drag!
> >
> And it leaves a bad taste in the mouth that perhaps Perl doesn't
> welcome
> patches from newcomers.  

Ah!  You've discovered kid51's hidden agenda:  Getting contributions
from newcomers to P5P -- and reducing the fear of newcomers and their
possible mistakes on the part of current P5Pers!

> It's clear that we aren't really management
> types here.  A good manager will spend a lot of time making sure that
> things are set up to smooth the way for the workers; like Schwern
> seems
> to do (in my limited experience)

Thank you very much.
Jim Keenan

via perlbug:  queue: perl5 status: open

Thread Previous | Thread Next Perl Programming lists via nntp and http.
Comments to Ask Bjørn Hansen at | Group listing | About