Skip to content

Conversation

@alexanderkiel
Copy link
Member

@alexanderkiel alexanderkiel commented Aug 10, 2025

Had to change a lot due to sveltejs/kit#13999.

I replaced all usages of import { base } from '$app/paths'; with resolve. Resolve takes an existing RouteId with the corresponding params. The params are only typed the right way if supplied via props.

@alexanderkiel alexanderkiel self-assigned this Aug 10, 2025
@alexanderkiel alexanderkiel added the dependencies Pull requests that update a dependency file label Aug 10, 2025
@alexanderkiel alexanderkiel added this to the v1.1.0 milestone Aug 10, 2025
@codecov
Copy link

codecov bot commented Aug 11, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.37%. Comparing base (f21200c) to head (92107e8).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2887      +/-   ##
==========================================
- Coverage   95.37%   95.37%   -0.01%     
==========================================
  Files         392      392              
  Lines       24325    24325              
  Branches      742      741       -1     
==========================================
- Hits        23201    23199       -2     
- Misses        536      537       +1     
- Partials      588      589       +1     

see 2 files with indirect coverage changes

🚀 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.

@alexanderkiel alexanderkiel added this pull request to the merge queue Aug 12, 2025
Merged via the queue into main with commit eee9575 Aug 12, 2025
153 checks passed
@alexanderkiel alexanderkiel deleted the update-frontend branch August 12, 2025 07:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants