Skip to content

allow log inspection#8

Merged
l5yth merged 7 commits intomainfrom
l5y-lsu-logs
Feb 24, 2026
Merged

allow log inspection#8
l5yth merged 7 commits intomainfrom
l5y-lsu-logs

Conversation

@l5yth
Copy link
Copy Markdown
Owner

@l5yth l5yth commented Feb 24, 2026

  • allow log inspection
  • allow log inspection
  • allow log inspection

@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 24, 2026

Codecov Report

❌ Patch coverage is 96.75926% with 21 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/rows.rs 92.96% 18 Missing ⚠️
src/journal.rs 97.91% 2 Missing ⚠️
src/cli.rs 99.28% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 0bf2ae1b4d

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread src/app/tui/mod.rs Outdated
@l5yth l5yth merged commit 303debf into main Feb 24, 2026
5 checks passed
@l5yth l5yth deleted the l5y-lsu-logs branch February 24, 2026 14:58
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