-
Notifications
You must be signed in to change notification settings - Fork 55
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add kli import command for importing dot CESR file, add gateway role #921
Conversation
Signed-off-by: pfeairheller <[email protected]>
Signed-off-by: pfeairheller <[email protected]>
Signed-off-by: pfeairheller <[email protected]>
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.
Just noted some typos in the import command handler.
parser.add_argument("--file", help="File of streamed CESR events to import", required=True) | ||
|
||
|
||
def export(args): |
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.
Should be 'import'
|
||
""" | ||
|
||
ed = ImportDoer(name=args.name, |
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.
Should probably be id
or importDoer
?
|
||
super(ImportDoer, self).__init__(doers=doers) | ||
|
||
def exportDo(self, tymth, tock=0.0): |
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.
Should be importDo
|
||
|
||
def export(args): | ||
""" Command line list credential registries handler |
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.
Docstring is probably from some other functionality?
Tymist instance. Calling tymth() returns associated Tymist .tyme. | ||
tock (float): injected initial tock value | ||
|
||
Returns: doifiable Doist compatible generator method |
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 returns True
This PR includes: