-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
build: add type declarations to /dist
#517
Labels
enhancement
New feature or request
Comments
5 tasks
We could make use of // [vite.config.js](https://github.com/imgix/vue/blob/9aebc6f129422ed21a9272f1a7e3fd9b3207794e/vite.config.js#L41)
import dts from 'vite-plugin-dts';
// ...
export default defineConfig({
// ...
plugins: [vue(), dts({ rollupTypes: true })],
// generates imgix-vue.esm.d, imgix-vue.min.d
}) |
5 tasks
Assigned to @namack. Don't hesitate to reach out if you need any help/feedback tackling this! |
namack
added a commit
to namack/imgix-vue
that referenced
this issue
Dec 14, 2023
3 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Before you submit:
Is your feature request related to a problem? Please describe.
Type declaration
.d.ts
file should be included for each bundle in/dist
Describe the solution you'd like
Use a build step to place emitted declaration files into
/dist
Describe alternatives you've considered
Using the
package.json
types: "src/plugins/imgix-vue"
field. This is sub-optimal because the types are not bundled together.Additional context
package.json
fieldThe text was updated successfully, but these errors were encountered: