Skip to content
This repository has been archived by the owner on Oct 22, 2020. It is now read-only.

Improve Composer input_files call messaging #65

Open
dbandin opened this issue Jun 25, 2019 · 0 comments
Open

Improve Composer input_files call messaging #65

dbandin opened this issue Jun 25, 2019 · 0 comments
Labels
enhancement New feature or request

Comments

@dbandin
Copy link
Contributor

dbandin commented Jun 25, 2019

Current message:

2019-06-25 10:03:02,686 misc       INFO     Input None
<your string path to bna file/n>
2019-06-25 10:03:15,472 misc       WARNING  Replaced <default_path_to.bna> with <you_string_to.bna>

When executing composer, we get a logging info meessage saying "Input None" and it waits for input. You paste the path to the bna file and then it states that has replaced a default value with another value as a warning.

I think this is a good case where we can use printed to stdout messages and not the logger function, we don't need the input requested in the logs.

Example output could be:

2019-06-25 10:03:02,686 misc       INFO     Starting composer
2019-06-25 10:03:02,686 misc       INFO     Default bna file <path_to_default_bna>
### Please input full path to your BNA file (default: /root/[email protected]) ###
### insert path: _ <- You wait for input here


2019-06-25 10:03:15,472 misc       INFO  Processing bna file -> /defined_path/file.bna
@dbandin dbandin added the enhancement New feature or request label Jun 25, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant