Skip to content

fix: end top-level span at semicolons#136

Merged
shssoichiro merged 1 commit into
shssoichiro:masterfrom
Huliiiiii:issue-130
Jul 2, 2026
Merged

fix: end top-level span at semicolons#136
shssoichiro merged 1 commit into
shssoichiro:masterfrom
Huliiiiii:issue-130

Conversation

@Huliiiiii

Copy link
Copy Markdown
Contributor

Close #130

@kilo-code-bot

kilo-code-bot Bot commented Jun 19, 2026

Copy link
Copy Markdown

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (2 files)
  • src/formatter.rs
  • src/lib.rs
Previous Review Summaries (2 snapshots, latest commit ada9f70)

Current summary above is authoritative. Previous snapshots are kept for context only.

Previous review (commit ada9f70)

Status: No Issues Found | Recommendation: Merge

Files Reviewed (2 files)
  • src/lib.rs
  • src/formatter.rs (unchanged; previous findings carried forward)

Previous review (commit a669f75)

Status: No Issues Found | Recommendation: Merge

Files Reviewed (2 files)
  • src/formatter.rs
  • src/lib.rs

Reviewed by kimi-k2.7-code-20260612 · Input: 28.5K · Output: 2.9K · Cached: 173.3K

@mergify

mergify Bot commented Jul 2, 2026

Copy link
Copy Markdown

Tick the box to add this pull request to the merge queue (same as @mergifyio queue).

  • Queue this pull request

@shssoichiro shssoichiro merged commit 90c1eaf into shssoichiro:master Jul 2, 2026
8 checks passed
@shssoichiro

Copy link
Copy Markdown
Owner

Thanks!

@codecov

codecov Bot commented Jul 2, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 95.23810% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 99.20%. Comparing base (56903e6) to head (eb63416).
⚠️ Report is 4 commits behind head on master.

Files with missing lines Patch % Lines
src/formatter.rs 75.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #136      +/-   ##
==========================================
- Coverage   99.22%   99.20%   -0.03%     
==========================================
  Files           7        7              
  Lines        4024     4045      +21     
  Branches     4024     4045      +21     
==========================================
+ Hits         3993     4013      +20     
- Misses         20       21       +1     
  Partials       11       11              

☔ View full report in Codecov by Harness.
📢 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.

@Huliiiiii Huliiiiii deleted the issue-130 branch July 2, 2026 06:15
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.

When a DROP TABLE statement is followed by a comment, the statement gets split onto a new line

2 participants