Page MenuHomeFreeBSD

New port devel/py-delfick_error

Authored by truckman on Feb 9 2016, 10:04 AM.



The point of this exception class is to be able to create
an error class that automatically combines keyword arguments
given to the exception instance.

Diff Detail

rP FreeBSD ports repository
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

truckman retitled this revision from to New port devel/py-delfick_error.
truckman updated this object.
truckman edited the test plan for this revision. (Show Details)
truckman added a reviewer: koobs.
koobs edited edge metadata.

Inline suggestion to use direct test invocation (rather than through script)

Otherwise LGTM.

43 ↗(On Diff #13145)

Prefer to invoke tests directly

We'll be providing framework variables for this soon, so it will assist in the conversion process

Same ordered do-test recommended as the previous review

Your call if you want to change this or not, but highly desirable.

This revision is now accepted and ready to land.Feb 9 2016, 10:08 AM

It is too late for me to commit this tonight ...

been on front of this screen for most of the last 20 hours ... ZZZ ...

43 ↗(On Diff #13145)

cd /wrkdirs/usr/ports/devel/py-delfick_error/work/delfick_error-6761e6c && /usr/local/bin/python2.7

File "", line 2
  nosetests --with-noy $@

SyntaxError: invalid syntax

truckman edited edge metadata.

Use preferred do-test construct

This revision now requires review to proceed.Feb 9 2016, 5:37 PM
koobs edited edge metadata.
This revision is now accepted and ready to land.Feb 9 2016, 5:51 PM

I believe GH_PROJECT defaults to PORTNAME and is not needed

This revision was automatically updated to reflect the committed changes.