Skip to content

Migrate to eslint-plugin-import-x AND enable it - #1859

Open
BotBlake wants to merge 4 commits into
jellyfin:masterfrom
BotBlake:eslint-plugin-import
Open

Migrate to eslint-plugin-import-x AND enable it#1859
BotBlake wants to merge 4 commits into
jellyfin:masterfrom
BotBlake:eslint-plugin-import

Conversation

@BotBlake

Copy link
Copy Markdown
Member

Fixes #1856

eslint-plugin-import was not configured to run AND is not maintained to our liking.
This migrates to eslint-plugin-import-x, a community maintained fork.

ALSO this PR configures the plugin to actually do something.
Which lead to a bunch of issues popping up that are fixed as well.

The only unresolved issue at the moment is resolving the docusaurus aliases.

@BotBlake

Copy link
Copy Markdown
Member Author

@jellyfin-bot Hey where the heck is my preview :(

@BotBlake
BotBlake force-pushed the eslint-plugin-import branch from 32cf9a6 to 526dd59 Compare May 12, 2026 12:23
@BotBlake BotBlake mentioned this pull request May 12, 2026
1 task
@felix920506

Copy link
Copy Markdown
Member

your lint is failing 🗿

@BotBlake

Copy link
Copy Markdown
Member Author

Yes, I am waiting for Violet to fix it.
This PR does enable the CI that was never enabled. That part works.

I have no idea how to fix the typescript code though and Violet promised weeks ago to have a look at it.

@BotBlake
BotBlake force-pushed the eslint-plugin-import branch 2 times, most recently from 66001f5 to ca48f6d Compare August 5, 2026 15:31
@BotBlake
BotBlake requested a review from a team as a code owner August 5, 2026 15:31
@BotBlake
BotBlake force-pushed the eslint-plugin-import branch from ca48f6d to 9f047c5 Compare August 5, 2026 15:38
@jellyfin-bot

Copy link
Copy Markdown

Cloudflare Pages deployment

Latest commit 9f047c57d65482afd48eb67d0afec5727d21cc2a
Status ✅ Deployed!
Preview URL https://a97e7831.jellyfin-org.pages.dev
Type 🔀 Preview

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.

Migrate from eslint-plugin-import

3 participants