Skip to content

Move SystemLog to neo-cli#587

Merged
shargon merged 22 commits intoneo-project:masterfrom
joeqian10:fix-neo-modules-228
May 26, 2020
Merged

Move SystemLog to neo-cli#587
shargon merged 22 commits intoneo-project:masterfrom
joeqian10:fix-neo-modules-228

Conversation

@joeqian10
Copy link
Copy Markdown
Contributor

@joeqian10 joeqian10 commented May 15, 2020

Close #228.

@joeqian10 joeqian10 requested a review from erikzhang May 15, 2020 10:40
@joeqian10
Copy link
Copy Markdown
Contributor Author

Help needed:
image

@Tommo-L
Copy link
Copy Markdown
Contributor

Tommo-L commented May 20, 2020

Waiting for neo-project/neo#1656

Co-authored-by: Luchuan <luchuan@ngd.neo.org>
shargon
shargon previously approved these changes May 21, 2020
@erikzhang
Copy link
Copy Markdown
Member

Should we remove the --mainnet and --testnet options first?

@vncoelho
Copy link
Copy Markdown
Member

I think it is good to remove as you suggested and discussed above, @erikzhang.

@vncoelho
Copy link
Copy Markdown
Member

However, @erikzhang, it is now blocking neo-project/neo-modules#246 (review).
Perhaps better to merge and open a new PR.

vncoelho
vncoelho previously approved these changes May 21, 2020
@shargon shargon dismissed stale reviews from vncoelho and themself via b1355e4 May 21, 2020 18:41
shargon
shargon previously approved these changes May 21, 2020
Copy link
Copy Markdown
Contributor

@nicolegys nicolegys left a comment

Choose a reason for hiding this comment

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

Test passed.

Situations:
1."ConsoleOutput": false, "Active": false
2."ConsoleOutput": false, "Active": true
3."ConsoleOutput": true, "Active": true

Folders in Logs_xxx:
image

@shargon shargon merged commit ecc0110 into neo-project:master May 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Move SystemLog to a neo-cli command

7 participants