Skip to content

chore: run format#142

Merged
DylanPiercey merged 1 commit into
mainfrom
format-fix
Jun 14, 2026
Merged

chore: run format#142
DylanPiercey merged 1 commit into
mainfrom
format-fix

Conversation

@DylanPiercey

Copy link
Copy Markdown
Contributor

No description provided.

@DylanPiercey DylanPiercey merged commit 8f3f49d into main Jun 14, 2026
1 check passed
@DylanPiercey DylanPiercey deleted the format-fix branch June 14, 2026 18:58
@coderabbitai

coderabbitai Bot commented Jun 14, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 244693aa-a93b-4dec-b3fc-21a6f3ebbbde

📥 Commits

Reviewing files that changed from the base of the PR and between c26ed5a and 8516a8d.

📒 Files selected for processing (2)
  • src/tags/app-header/app-header.marko
  • src/tags/search-dialog/search-dialog.marko

Walkthrough

Both app-header.marko and search-dialog.marko receive formatting-only refactors. In app-header.marko, the Cmd/Ctrl+K document.addEventListener call is expanded to a multi-line arrow-function body, and the platform-specific shortcut html-script template is rewrapped into a multi-line ${ ... } block. In search-dialog.marko, highlightSegments adopts a multi-line filter/map chain, results gains an explicit SearchHit[] | undefined cast, the ArrowDown/ArrowUp keyboard handlers are expanded to multi-line blocks, a redundant parentheses pair is removed from the results condition, and search result rendering for titleMatch/displayText and seg.hl markup is reformatted. No behavioral or output changes are introduced.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch format-fix

Comment @coderabbitai help to get the list of available commands and usage tips.

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