-
Notifications
You must be signed in to change notification settings - Fork 951
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
Ability to overwrite to log file instead of only appending #5
Comments
Nice idea, feel free to submit a pull request. Probably just truncate the
file when the function begins.
…On Mon, Apr 22, 2019, 16:11 Kandeel4411 ***@***.***> wrote:
Would be nice if we could overwrite existing log files
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#5>, or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAAN3SSPDERQNSCVNUOGOPTPRW2RFANCNFSM4HHPTSAQ>
.
|
If you're submitting a pull request, I'd require it to include tests.
…On Mon, Apr 22, 2019, 16:17 Ram Rachum ***@***.***> wrote:
Nice idea, feel free to submit a pull request. Probably just truncate the
file when the function begins.
On Mon, Apr 22, 2019, 16:11 Kandeel4411 ***@***.***> wrote:
> Would be nice if we could overwrite existing log files
>
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly, view it on GitHub
> <#5>, or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AAAN3SSPDERQNSCVNUOGOPTPRW2RFANCNFSM4HHPTSAQ>
> .
>
|
Can't you just open the file with a different mode? 'w' instead of 'a'? |
No because I reopen on every write.
…On Mon, Apr 22, 2019, 16:58 Patrick Robertshaw ***@***.***> wrote:
Can't you just open the file with a different mode? 'w' instead of 'a'?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#5 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAAN3SX62TUDBEWZLTQENA3PRXAABANCNFSM4HHPTSAQ>
.
|
I feel this idea "Ability to overwrite to log file instead of only appending" is very nice, but after I clone the repo today, unfortunately, I found this idea has not implemented. |
@836304831 feel free to submit a pull request |
cool-RR
added a commit
that referenced
this issue
Apr 25, 2019
cool-RR
added a commit
that referenced
this issue
Apr 25, 2019
cool-RR
added a commit
that referenced
this issue
Apr 25, 2019
cool-RR
added a commit
that referenced
this issue
Apr 25, 2019
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Would be nice if we could overwrite existing log files
The text was updated successfully, but these errors were encountered: