Skip to content

feat(log): add an info level to NotificationLog - #38

Closed
anilcancakir wants to merge 1 commit into
masterfrom
kodizm-review-test/log-info
Closed

anilcancakir wants to merge 1 commit into
masterfrom
kodizm-review-test/log-info

Conversation

@anilcancakir

Copy link
Copy Markdown
Member

Test pull request for the Kodizm automatic review pipeline (parallel review measurement). It will not be merged: it is closed and its branch deleted once the measurement is done.

@codecov

codecov Bot commented Sep 23, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@kodizm

kodizm Bot commented Sep 23, 2026

Copy link
Copy Markdown

Note

Kodizm (AI-generated). May contain mistakes; verify before acting.

The change looks correct: info() follows the same guard-then-delegate shape as the error, warning and debug methods next to it.

Minor

CHANGELOG.md — maintainability: CLAUDE.md's Post-Change Checklist requires an entry under [Unreleased] after any source change, and this PR adds none. Nothing in the diff uses info() yet, so it is an unused API addition. If the PR is merged, add the entry.

Tests

Covers the bound case (level and message, exactly one entry) and the unbound case (returnsNormally after Magic.flush()), mirroring the existing tests for the other levels.

CI

  • codecov/patch: success (100% of diff hit)
  • Lint & Test: success
  • Auto-merge low-risk Dependabot PRs: skipped

@anilcancakir

Copy link
Copy Markdown
Member Author

Measurement done; closing as announced. Not for merge.

@anilcancakir
anilcancakir deleted the kodizm-review-test/log-info branch September 23, 2026 17:36
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.

1 participant