Skip to content

Conversation

@tylers-username
Copy link
Contributor

@tylers-username tylers-username commented Dec 12, 2025

A follow up to #6248

ℹ️ Note This MR includes #7421. #7421 should be merged first for a cleaner review of this MR.

Changes

  • Updates webpack, tsconfig, and eslint to enforce the use of @alias paths with the exclusion of same-directory imports such as ./
  • Executed lint-fix to apply changes to relative paths
  • Executed lint to identify imports that fail to resolve
  • Ran build es-check to review changes

@jellyfin-bot
Copy link
Collaborator

jellyfin-bot commented Dec 12, 2025

Cloudflare Pages deployment

Latest commit ed77c76
Status ✅ Deployed!
Preview URL https://40b3808d.jellyfin-web.pages.dev
Type 🔀 Preview

View build logs

@jellyfin-bot jellyfin-bot added the merge conflict Conflicts prevent merging label Dec 12, 2025
@jellyfin-bot
Copy link
Collaborator

This pull request has merge conflicts. Please resolve the conflicts so the PR can be successfully reviewed and merged.

@sonarqubecloud
Copy link

@tylers-username
Copy link
Contributor Author

This could cause conflicts with much of the backlog because many files in this MR have been updated to pass the @/ alias check.

The suggested way forward (Discord discussion) is to consider merging the base MR #6248 into production. Meanwhile, keep this MR open so we can use the "Update Branch" feature to keep it ready for merging.

@thornbill thornbill added the cleanup Cleanup of legacy code or code smells label Dec 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cleanup Cleanup of legacy code or code smells

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants