-
-
Notifications
You must be signed in to change notification settings - Fork 21
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
Is there a verbose execution mode? #1
Comments
Hi @lestephane, yes, it is definitely needed, I'll work on this when I get some time. Thinking aloud here: perhaps a single I'll see what the command-line options library allow here. |
@lestephane I've started a branch and a pull request to work on this: Currently the import subcommand will accept a verbose flag on that branch, but it doesn't use the flag just yet. |
Hi @lestephane, I've added some verbose output on the Could you try it out? This is an example of what you might see:
|
I changed the output to include the execution times of the custom scripts and a few other tweaks:
|
Optional verbose mode is now in master, after #2 has been merged |
Thanks, I'll try it out!
…On Tue, Mar 5, 2019 at 12:03 AM Andreas Pauley ***@***.***> wrote:
Closed #1 <#1>.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#1 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AFNTlHltWABNw4PoOHemEPBVfvdKnP2Nks5vTZg5gaJpZM4bXtcT>
.
|
@lestephane There has been another merge that might affect you if you already got to a point where you created an opening journal. The file needs to move one level up now. The PR description has more details: #3 |
Ok thanks, I still haven't gotten to it yet
…On Fri, Mar 15, 2019 at 8:54 AM Andreas Pauley ***@***.***> wrote:
@lestephane <https://github.com/lestephane> There has been another merge
that might affect you if you already got to a point where you created an
opening journal. The file needs to move one level up now.
The PR description has more details: #3
<#3>
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#1 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AFNTlNmJiG69Y4QZMgc5OM6if8-GO0I7ks5vW0O0gaJpZM4bXtcT>
.
|
I'd like to see some output when running
hledger-makeitso import
Background:
hledger-makeitso import
repeatedly failed for me when I tried makeitso for the first time today. I eventually figured that I called my preprocess script preprocessor instead of preprocessThe text was updated successfully, but these errors were encountered: