Skip to content
This repository has been archived by the owner on Dec 7, 2023. It is now read-only.

Move SystemLog plugin into neo-cli as a native logger with on/off functionalities. #582

Merged
merged 7 commits into from
May 13, 2020

Conversation

joeqian10
Copy link
Contributor

@joeqian10 joeqian10 commented May 12, 2020

Close 228.

neo-cli/CLI/MainService.Node.cs Outdated Show resolved Hide resolved
neo-cli/CLI/MainService.cs Outdated Show resolved Hide resolved
neo-cli/SystemLog/Logger.cs Outdated Show resolved Hide resolved
neo-cli/CLI/Logger.cs Outdated Show resolved Hide resolved
@superboyiii
Copy link
Member

@joeqian10 Unhandled exceptions are not recorded in System log.
image
Now:
image
Before:
image

Co-authored-by: Owen Zhang <38493437+superboyiii@users.noreply.github.com>
neo-cli/CLI/Logger.cs Outdated Show resolved Hide resolved
Copy link
Member

@superboyiii superboyiii left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Test: PASS
All types of errors is recorded normally.

@superboyiii superboyiii merged commit 21a4a5b into neo-project:master May 13, 2020
@erikzhang
Copy link
Member

erikzhang commented May 14, 2020

Why merge it so quickly? I haven't finished my review.

@erikzhang
Copy link
Member

Reverted.

erikzhang added a commit that referenced this pull request May 14, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Move SystemLog to a neo-cli command
4 participants