You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The reason will be displayed to describe this comment to others. Learn more.
It seems to be that it would be better to have a __main__.py file. Then, from the top level of the git repo, it would be as simple as running python alot.
The reason will be displayed to describe this comment to others. Learn more.
@pazz No, that is what would happen if the code was but in __init__.py. __main__.py is run when the module is executed as a script, so it should not cause any such problem.
70d0bc3
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems to be that it would be better to have a
__main__.py
file. Then, from the top level of the git repo, it would be as simple as runningpython alot
.70d0bc3
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wouldn't that start the ui even if i just
import alot.foo
? Otherwise,__main__.py
might be a good idea70d0bc3
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@pazz No, that is what would happen if the code was but in
__init__.py
.__main__.py
is run when the module is executed as a script, so it should not cause any such problem.70d0bc3
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
right. let's have a
__main__.py
then :)70d0bc3
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ids1024 : does this look OK to you? https://github.com/pazz/alot/tree/0.3.6-refactor-main
70d0bc3
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I do this a bit differently: I install packages I want to tinker on with
cd path/to/alot && pip install -e .
and can than just runalot
.70d0bc3
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@pazz Yes, that looks ok to me.