Skip to content

Use $app/state instead of $app/stores#1084

Merged
quentinderoubaix merged 1 commit into
AmadeusITGroup:mainfrom
quentinderoubaix:advent-of-svelte
Feb 19, 2026
Merged

Use $app/state instead of $app/stores#1084
quentinderoubaix merged 1 commit into
AmadeusITGroup:mainfrom
quentinderoubaix:advent-of-svelte

Conversation

@quentinderoubaix
Copy link
Copy Markdown
Contributor

No description provided.

@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 6, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.88%. Comparing base (8cfb531) to head (abe5cbf).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1084   +/-   ##
=======================================
  Coverage   90.88%   90.88%           
=======================================
  Files         118      118           
  Lines        3456     3456           
  Branches      554      554           
=======================================
  Hits         3141     3141           
  Misses        204      204           
  Partials      111      111           
Flag Coverage Δ
e2e 81.78% <ø> (ø)
unit 87.12% <ø> (ø)

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:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@quentinderoubaix quentinderoubaix marked this pull request as draft January 6, 2025 20:37
@quentinderoubaix quentinderoubaix force-pushed the advent-of-svelte branch 2 times, most recently from 888f63d to cd7278f Compare January 7, 2025 09:02
@quentinderoubaix quentinderoubaix marked this pull request as ready for review January 7, 2025 09:17
@quentinderoubaix quentinderoubaix changed the title Use $app/state instead of $app/stores + createSubscriber in coffee brewer demo Use $app/state instead of $app/stores Jan 7, 2025
@quentinderoubaix quentinderoubaix marked this pull request as draft January 7, 2025 12:31
@quentinderoubaix
Copy link
Copy Markdown
Contributor Author

Let's wait for clarification from sveltejs/svelte#14954

@quentinderoubaix quentinderoubaix marked this pull request as ready for review February 19, 2026 11:21
@quentinderoubaix quentinderoubaix merged commit 2a1c0da into AmadeusITGroup:main Feb 19, 2026
16 checks passed
@quentinderoubaix quentinderoubaix deleted the advent-of-svelte branch February 19, 2026 11:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant