Skip to content

Improve the log-level setting documentation#490

Merged
pajlada merged 3 commits intomasterfrom
doc/log-level
Jun 11, 2023
Merged

Improve the log-level setting documentation#490
pajlada merged 3 commits intomasterfrom
doc/log-level

Conversation

@pajlada
Copy link
Copy Markdown
Member

@pajlada pajlada commented Jun 11, 2023

Pull request checklist:

  • CHANGELOG.md was updated, if applicable

Description

  • Add log-level documentation to config.yaml
  • Improve --help output for --log-level

Works on #489 but doesn't fully fix it

@pajlada pajlada changed the title Improve log-level config documentation Improve the log-level setting documentation Jun 11, 2023
@pajlada pajlada requested a review from zneix June 11, 2023 10:30
@pajlada pajlada marked this pull request as ready for review June 11, 2023 10:30
@codecov
Copy link
Copy Markdown

codecov bot commented Jun 11, 2023

Codecov Report

Merging #490 (4f1d0b6) into master (a2021b3) will not change coverage.
The diff coverage is 100.00%.

❗ Current head 4f1d0b6 differs from pull request most recent head 2796e4b. Consider uploading reports for the commit 2796e4b to get more accurate results

@@           Coverage Diff           @@
##           master     #490   +/-   ##
=======================================
  Coverage   46.90%   46.90%           
=======================================
  Files          99       99           
  Lines        3710     3710           
=======================================
  Hits         1740     1740           
  Misses       1920     1920           
  Partials       50       50           
Impacted Files Coverage Δ
pkg/config/config.go 41.05% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@pajlada pajlada enabled auto-merge (squash) June 11, 2023 11:08
@pajlada pajlada merged commit 7385136 into master Jun 11, 2023
@pajlada pajlada deleted the doc/log-level branch June 11, 2023 11:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants