Skip to content

chore: use unplugin-dts in favour of vite-plugin-dts #706

Open
dopenguin wants to merge 1 commit intonextfrom
chore/update-dts-plugin
Open

chore: use unplugin-dts in favour of vite-plugin-dts #706
dopenguin wants to merge 1 commit intonextfrom
chore/update-dts-plugin

Conversation

@dopenguin
Copy link
Copy Markdown
Member

As recommended, use unplugin-dts in favour of vite-plugin-dts.

Relevant tickets, issues, et cetera

Closes: #700

@dopenguin dopenguin added this to the POLAR@3 milestone May 5, 2026
@dopenguin dopenguin self-assigned this May 5, 2026
@dopenguin dopenguin added the dependencies Pull requests that update a dependency file label May 5, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 5, 2026

PR Preview Action v1.8.1

QR code for preview link

🚀 View preview at
https://Dataport.github.io/polar/pr-preview/pr-706/

Built to branch gh-pages at 2026-05-11 10:08 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

Copy link
Copy Markdown
Collaborator

@oeninghe-dataport oeninghe-dataport left a comment

Choose a reason for hiding this comment

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

Looks good to be.

The docs state that, for Vue projects, @vue/language-core should be installed and processor: 'vue' should be set. Is there a reason to not do this?

@dopenguin
Copy link
Copy Markdown
Member Author

Looks good to be.

The docs state that, for Vue projects, @vue/language-core should be installed and processor: 'vue' should be set. Is there a reason to not do this?

The version wanted by unplugin-dts is in conflict with the version wanted by vue-tsc.

Bildschirmfoto 2026-05-06 um 11 04 58

Also, this is the TSDoc, so not sure if this is actually necessary:
Bildschirmfoto 2026-05-06 um 11 07 26

@dopenguin dopenguin force-pushed the chore/update-dts-plugin branch from 15fbe33 to 18bb72a Compare May 11, 2026 10:06
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.

2 participants