Skip to content

chore: extract LiveReloadMiddleware to a separate file#375

Merged
seqre merged 1 commit into
masterfrom
extract-live-reload
Jul 7, 2025
Merged

chore: extract LiveReloadMiddleware to a separate file#375
seqre merged 1 commit into
masterfrom
extract-live-reload

Conversation

@m4tx

@m4tx m4tx commented Jul 7, 2025

Copy link
Copy Markdown
Member

The module still reexports the type at the same place, but the middleware has been extracted to make quite a big file a little bit smaller.

Also includes some basic tests for the middleware and tiny doc fixes.

@m4tx m4tx requested a review from seqre July 7, 2025 16:40
@github-actions github-actions Bot added the C-lib Crate: cot (main library crate) label Jul 7, 2025
The module still reexports the type at the same place, but the
middleware has been extracted to make quite a big file a little bit
smaller.

Also includes some basic tests for the middleware and tiny doc fixes.
@m4tx m4tx force-pushed the extract-live-reload branch from f4db9a5 to 643b7bd Compare July 7, 2025 16:56
@codecov

codecov Bot commented Jul 7, 2025

Copy link
Copy Markdown

Codecov Report

Attention: Patch coverage is 84.00000% with 8 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
cot/src/middleware/live_reload.rs 84.00% 6 Missing and 2 partials ⚠️
Flag Coverage Δ
rust 89.31% <84.00%> (+0.04%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
cot/src/middleware.rs 85.18% <ø> (+3.33%) ⬆️
cot/src/middleware/live_reload.rs 84.00% <84.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@seqre seqre merged commit f9a0945 into master Jul 7, 2025
35 of 36 checks passed
@seqre seqre deleted the extract-live-reload branch July 7, 2025 17:42
@cotbot cotbot Bot mentioned this pull request Jul 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C-lib Crate: cot (main library crate)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants