Code review comment for lp:~maphew/leo-editor/pypi-packaging

Revision history for this message
Jake Peck (gatesphere) wrote :

On 5/10/2013 7:25 AM, Edward K. Ream wrote:
> On Fri, May 10, 2013 at 3:55 AM, Matt Wilkie <email address hidden> wrote:
>
>> Matt Wilkie has proposed merging lp:~maphew/leo-editor/pypi-packaging into
>> lp:leo-editor.
>>
> Go ahead. The script will be a starting point. Not sure what I think
> about .leos (Leo script) file extension: it's just a .py file.
>
> Edward
>
I agree, w/r/t the .leos extension - just do .py, and perhaps have a
guard in the `if __name__ == '__main__':` bit... perhaps check for the
existence of g.app? If it's not found, print a message stating that
"This script needs to be run from Leo!" and exit.

-->Jake

« Back to merge proposal