WIP : added --silent-mode and --fxlink-log options #7

Closed
Slyvtt wants to merge 0 commits from Slyvtt/fxsdk:dev into dev
Contributor

Hi Lephé

Please find the proposal for adding the following options to fxlink :

silent-mode : limit the verbosity of the output when sending text

fxlink-log(=filename) : log the text output to a file (if filename is given, it will create/append the file, if not it will create a file called fxlink-logfile(+date/timestamp).log in the current directory.

Regards

SlyVTT

Hi Lephé Please find the proposal for adding the following options to fxlink : silent-mode : limit the verbosity of the output when sending text fxlink-log(=filename) : log the text output to a file (if filename is given, it will create/append the file, if not it will create a file called fxlink-logfile(+date/timestamp).log in the current directory. Regards SlyVTT
Slyvtt added 1 commit 2022-04-19 18:44:26 +02:00
Owner

Hi! That's the idea. There are many small details to adjust, but that's for me to do I guess.

Note that for the silent mode we should use -q, --quiet as it's a fairly standard option.

Apart from that I guess the external interface shouldn't change.

Hi! That's the idea. There are many small details to adjust, but that's for me to do I guess. Note that for the silent mode we should use `-q, --quiet` as it's a fairly standard option. Apart from that I guess the external interface shouldn't change.
Author
Contributor

Hey :D

Good. I can make the change for the '-q, --quiet' option this is not a problem, just let me know.

Hey :D Good. I can make the change for the '-q, --quiet' option this is not a problem, just let me know.
Owner

Sure, go ahead. This way you'll have the full interface in advance. The other things are just boring stuff: factoring, brace placement, column limit, etc; don't worry about it.

Sure, go ahead. This way you'll have the full interface in advance. The other things are just boring stuff: factoring, brace placement, column limit, etc; don't worry about it.
Author
Contributor

Ok. Just realized I haven't followed the existing coding rules (braces, ...). I will also improve this when changing the --quiet option.

Ok. Just realized I haven't followed the existing coding rules (braces, ...). I will also improve this when changing the --quiet option.
Slyvtt added 1 commit 2022-04-20 13:08:16 +02:00
Slyvtt closed this pull request 2022-04-20 13:09:11 +02:00

Pull request closed

Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: Lephenixnoir/fxsdk#7
No description provided.