Skip to content

Add typescript definition file #13

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Closed
wants to merge 3 commits into from
Closed

Conversation

akuma06
Copy link

@akuma06 akuma06 commented Oct 15, 2018

I was using this library for a personal app and I needed a typescript definition file so here it is!

Since it's a small code base and there is not much properties going on, every vue properties are declared for ContentLoader and the other definitions are just extending ContentLoaderConstructor.

Already tested on my local source code, need to be tested before merged though (first time I'm doing a .d.ts file for a particular module).

Should solve #11

@egoist
Copy link
Owner

egoist commented Oct 15, 2018

Deploy preview for vue-content-loader ready!

Built with commit 510678d

https://deploy-preview-13--vue-content-loader.netlify.com

@egoist
Copy link
Owner

egoist commented Oct 16, 2018

Is it better to be submitted to DefinitelyTyped? ok, i think it's fine to be kept here.

@@ -7,6 +7,7 @@
"type": "git"
},
"main": "dist/vue-content-loader.cjs.js",
"types": "src/vue-content-loader.d.ts",
Copy link
Owner

Choose a reason for hiding this comment

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

maybe move to index.d.ts? src folder is not published on npm.

LeBenLeBen added a commit that referenced this pull request Jan 27, 2021
@LeBenLeBen
Copy link
Collaborator

Closing in favor of #121

@LeBenLeBen LeBenLeBen closed this Jan 27, 2021
LeBenLeBen added a commit that referenced this pull request Jan 27, 2021
@egoist
Copy link
Owner

egoist commented Jan 27, 2021

🎉 This issue has been resolved in version 2.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants