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.

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

log: add silent mode #93

Merged
merged 3 commits into from
Aug 6, 2022
Merged

log: add silent mode #93

merged 3 commits into from
Aug 6, 2022

Conversation

k3rn31
Copy link
Member

@k3rn31 k3rn31 commented Aug 3, 2022

In this PR I added a silent mode. It can be enabled via --silent/-s. In this mode, gremlins will be completely silent and just report errors if they happen.

@k3rn31 k3rn31 added the c/feature Category: An issue or PR related to a new feature label Aug 3, 2022
@k3rn31 k3rn31 added this to the v0.2.0 milestone Aug 3, 2022
@k3rn31 k3rn31 self-assigned this Aug 3, 2022
@k3rn31 k3rn31 linked an issue Aug 3, 2022 that may be closed by this pull request
@pull-request-size pull-request-size bot added the s/M Size: Denotes a Pull Request that changes 30-99 lines label Aug 3, 2022
@k3rn31 k3rn31 force-pushed the 73-allow-to-silence-stdout-logs branch 2 times, most recently from 5835320 to 0d82e3b Compare August 3, 2022 15:56
@codecov
Copy link

codecov bot commented Aug 3, 2022

Codecov Report

Merging #93 (d6ef943) into main (3cff58c) will decrease coverage by 0.07%.
The diff coverage is 86.36%.

@@            Coverage Diff             @@
##             main      #93      +/-   ##
==========================================
- Coverage   86.11%   86.03%   -0.08%     
==========================================
  Files          16       16              
  Lines         864      881      +17     
==========================================
+ Hits          744      758      +14     
- Misses         95       97       +2     
- Partials       25       26       +1     
Flag Coverage Δ
unittests 86.03% <86.36%> (-0.08%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
configuration/configuration.go 94.44% <ø> (ø)
cmd/gremlins.go 71.73% <25.00%> (-4.46%) ⬇️
cmd/internal/flags/flags.go 100.00% <100.00%> (ø)
pkg/log/log.go 93.75% <100.00%> (+0.89%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@k3rn31 k3rn31 force-pushed the 73-allow-to-silence-stdout-logs branch from 0d82e3b to e7d8a81 Compare August 3, 2022 16:07
@pull-request-size pull-request-size bot added s/L Size: Denotes a Pull Request that changes 100-499 lines and removed s/M Size: Denotes a Pull Request that changes 30-99 lines labels Aug 3, 2022
@k3rn31 k3rn31 force-pushed the 73-allow-to-silence-stdout-logs branch 5 times, most recently from 03fa72d to 436173c Compare August 5, 2022 04:48
@k3rn31 k3rn31 force-pushed the 73-allow-to-silence-stdout-logs branch from 413579d to 5ed13b3 Compare August 5, 2022 15:25
@codeclimate
Copy link

codeclimate bot commented Aug 6, 2022

Code Climate has analyzed commit d6ef943 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 91.6% (70% is the threshold).

This pull request will bring the total coverage in the repository to 88.9% (0.0% change).

View more on Code Climate.

@k3rn31 k3rn31 merged commit 9c863e8 into main Aug 6, 2022
@k3rn31 k3rn31 deleted the 73-allow-to-silence-stdout-logs branch August 6, 2022 05:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c/feature Category: An issue or PR related to a new feature s/L Size: Denotes a Pull Request that changes 100-499 lines
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

Allow to silence stdout logs
1 participant