-
Notifications
You must be signed in to change notification settings - Fork 598
Use click instead of argparse in py instance #3598
Conversation
9636663
to
bcb20fc
Compare
0a51882
to
d7aea32
Compare
d7aea32
to
b6aa44d
Compare
Tested on Ubuntu20.04 The job can run. The heron-instance logs look good to me. Found some warning in heron-executor log:
|
This is a clean up which also avoids [this kind of runtime warning](https://stackoverflow.com/questions/43393764/python-3-6-project-structure-leads-to-runtimewarning).
Thanks @huijunwu! I just pushed a change to avoid triggering that existing issue, which hopefully wasn't causing anything or relied on. An example of what causes that
|
@Code0x58 is this ready to be re-reviewed? |
Yep. I think it's fair to assume that @huijunwu's check still holds, then the change on top to stop the logs warning should be ok (test suite still passes, although it could have in theory caused issues), so could call it good for merge as is |
No description provided.