Skip to content

Conversation

KyleAMathews
Copy link
Contributor

No description provided.

Copy link

codecov bot commented Sep 24, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.20%. Comparing base (5ca7997) to head (ff2739a).
⚠️ Report is 14 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3170      +/-   ##
==========================================
- Coverage   77.05%   76.20%   -0.86%     
==========================================
  Files         175      176       +1     
  Lines        9263     9321      +58     
  Branches      296      299       +3     
==========================================
- Hits         7138     7103      -35     
- Misses       2123     2216      +93     
  Partials        2        2              
Flag Coverage Δ
elixir 74.84% <ø> (-0.91%) ⬇️
elixir-client 74.47% <ø> (+0.59%) ⬆️
packages/experimental 87.73% <ø> (ø)
packages/react-hooks 86.48% <ø> (ø)
packages/typescript-client 92.54% <ø> (-0.96%) ⬇️
packages/y-electric 55.12% <ø> (ø)
postgres-140000 ?
postgres-150000 74.74% <ø> (?)
postgres-170000 74.86% <ø> (-1.08%) ⬇️
sync-service 74.88% <ø> (-1.06%) ⬇️
typescript 85.24% <ø> (-0.56%) ⬇️
unit-tests 76.20% <ø> (-0.86%) ⬇️

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Contributor

Examples

  • linearlite-read-only: deploy failed

Copy link
Contributor

@samwillis samwillis left a comment

Choose a reason for hiding this comment

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

LGTM - just a couple of llm comments to remove

Comment on lines +4 to +5
nodejs 22.12.0
pnpm 10.12.1
Copy link
Contributor

Choose a reason for hiding this comment

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

Did you intent to bump this for the whole monorepo at the same time?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah didn't realize you could nest these. I'll move the change to just the examples directory.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Actually that doesn't work as pnpm is still global 😆 so yes, I intended to do this.

Copy link
Contributor

Choose a reason for hiding this comment

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

Did you intent to make changes to the linerlite demo? (seems fine)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Part of the node version bump

Copy link
Contributor

Choose a reason for hiding this comment

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

Same comment, was this intended?

KyleAMathews and others added 2 commits September 29, 2025 14:02
Co-authored-by: Sam Willis <sam.willis@gmail.com>
Co-authored-by: Sam Willis <sam.willis@gmail.com>
Copy link

netlify bot commented Sep 29, 2025

Deploy Preview for electric-next ready!

Name Link
🔨 Latest commit ff2739a
🔍 Latest deploy log https://app.netlify.com/projects/electric-next/deploys/68dae71b0a506c0008a17199
😎 Deploy Preview https://deploy-preview-3170--electric-next.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@KyleAMathews KyleAMathews merged commit be1392f into main Sep 29, 2025
45 of 47 checks passed
@KyleAMathews KyleAMathews deleted the update-starter branch September 29, 2025 20: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.

2 participants