I’m not sure what you’re suggesting is vestigial. If you meant the comment line, it is not vestigial, but is there (along with its partner 3 lines earlier) to document that the various ‘send test event’ calls need a TestEventType parameter, but do NOT need a HITTypeId parameter. The reverse holds true for the  ‘set notification’ calls. If this is not what you meant, please let me know.

From: Daniel Lindsley [mailto:notifications@github.com]
Sent: Wednesday, April 03, 2013 5:45 PM
To: boto/boto
Cc: Dennis McRitchie
Subject: Re: [boto] Develop (#1389)


In boto/mturk/connection.py:

>          """

> -        params = {'HITTypeId': hit_type}

> +        # If SetHITTypeNotification, specify the HIT type

> +        if not test_event_type:

> +            params = {'HITTypeId': hit_type}

> +        # else if test notification, specify the notification type to be tested

Looks vestigial. Can this be removed?

—
Reply to this email directly or view it on GitHub<https://github.com/boto/boto/pull/1389/files#r3645850>.