Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Better error log #37

Merged
merged 7 commits into from
Aug 23, 2024
Merged

Better error log #37

merged 7 commits into from
Aug 23, 2024

Conversation

ggwpez
Copy link
Member

@ggwpez ggwpez commented Aug 1, 2024 •
edited
Loading

  • Set default log level to 'warn' to mute noise
  • Improve error message
  • Log full error messages instead of variant names
  • Bump all deps
  • Bump to 0.1.0

before:
Screenshot 2024-08-01 at 14 55 40

after:
Screenshot 2024-08-01 at 14 55 58

@ggwpez
Set default log level to 'warn' to mute noise
d5eea64 
Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
@ggwpez
Improve error message
7dd1cb2 
Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
@ggwpez
Log full error messages instead of variant names
98ecb8f 
Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
@ggwpez
Bump all deps
ea3d6cb 
Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
@ggwpez
Bump to 0.1.0
e8674a4 
Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
Morganamilo
Morganamilo reviewed Aug 1, 2024
View reviewed changes
cli/Cargo.toml Outdated Show resolved Hide resolved
@ggwpez
Fix version
c0bf7bf 
Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
@mutantcornholio mutantcornholio merged commit c70959a into master Aug 23, 2024
4 checks passed
@mutantcornholio mutantcornholio deleted the oty-logs branch August 23, 2024 16:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Morganamilo Morganamilo Morganamilo approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ggwpez @Morganamilo @mutantcornholio