Skip to content

Explained level options with examples#369

Merged
asmacdo merged 3 commits into
con:mainfrom
asmacdo:log-level-explained
May 11, 2026
Merged

Explained level options with examples#369
asmacdo merged 3 commits into
con:mainfrom
asmacdo:log-level-explained

Conversation

@asmacdo
Copy link
Copy Markdown
Member

@asmacdo asmacdo commented Jan 16, 2026

Closes #338

Copilot AI review requested due to automatic review settings January 16, 2026 18:13
@asmacdo asmacdo added the semver-documentation Changes only affect the documentation label Jan 16, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds comprehensive documentation for the --log-level option by providing a dedicated "Log Levels" section with usage guidance and practical examples. This addresses issue #338 by making it easier for users to understand when and how to use different log levels.

Changes:

  • Renamed "Command Reference" heading to "Reference" for brevity
  • Added "Log Levels" section with a table describing when to use each level
  • Provided three concrete examples showing INFO (default), NONE, and DEBUG output

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread README.md
@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 16, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.92%. Comparing base (0ef4edd) to head (915d715).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #369   +/-   ##
=======================================
  Coverage   91.92%   91.92%           
=======================================
  Files          15       15           
  Lines        1127     1127           
  Branches      140      140           
=======================================
  Hits         1036     1036           
  Misses         69       69           
  Partials       22       22           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@asmacdo asmacdo requested a review from CodyCBakerPhD February 2, 2026 18:11
Comment thread README.md Outdated
Control duct's verbosity with `--log-level` (or `DUCT_LOG_LEVEL` environment variable):

| Level | Use when... |
|-------|-------------|
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Recommendation: decide on alignment style

E.g.

Suggested change
|-------|-------------|
|:-------:|:-------------:|

aligns to center

Currently

Image

doesn't look quite as good as it 'could'

Up to you though

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

went with center for levels, left for the text. thanks!

@CodyCBakerPhD
Copy link
Copy Markdown
Contributor

'Explained level options with examples' for PR title (CHANGELOG)

One minor style suggestion

These docs are getting quite sophisticated

@asmacdo asmacdo changed the title explain level options with examples Explained level options with examples Feb 2, 2026
asmacdo and others added 3 commits May 11, 2026 09:42
@asmacdo asmacdo force-pushed the log-level-explained branch from 23fe39a to 915d715 Compare May 11, 2026 14:45
@asmacdo asmacdo merged commit aef263f into con:main May 11, 2026
29 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

semver-documentation Changes only affect the documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Enhance documentation with log level examples

3 participants