-
-
Notifications
You must be signed in to change notification settings - Fork 79
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
Typescript support #95
Comments
Yes, someday I will definitely add typescript support. Unfortunately, I can't give an estimate right now. Any help on this matter is welcome 😄 |
I have made progress on this. I switched the svelte packages to use svelte-package instead of rollup. I think this is a straight upgrade (as far as svelte support) from rollup, following svelte's/sveltekit's near-full migration to vite. In I originally was planning on just forking and maintaining my own publication, so I have made some other changes like using pnpm with workspaces, removing rollup/publishing from the top level, adding a GitHub workflow, and adding a linter/formatter. However, I found I didn't really want to mess with the internals (common), so I ended up just doing as much jsdoc as I could on top of your source thinking maybe merging would be possible. I hope at least you can find some use in the typing that I have added if merging seems like a problem. Let me know if you would like me to make a pr. I took out the scripts for versioning and publishing because I wanted to do that with changesets but never got around to it (or changing it back). I'd like to look into styling in the components instead of importing the scss stylesheet in Thank you for creating this package! You can see my fork here |
@ksmoore17 Thanks for starting this change. I looked at the result and it's impressive. I see value for the upstream repository in the types you defined here. I would not like to use jsdoc as an intermediate solution, but go straight to typescript. And I would not want to accept all your changes, because there are so many of them. Maybe the right way is a smooth transition to typescript by rewriting individual functions and components? So that no more than a few files get into the pull request at a time and for them it is possible to carefully do a code review? |
@vkurko Makes sense and sounds good to me. I will start rewriting |
More thoughts on typescript. Perhaps I was in a hurry with the fact that the entire project needs to be rewritten in typescript. After all, Svelte and SvelteKit projects that will use the Event Calendar will use the source files of the library. And if the project does not use typescript? It turns out an extra dependency that can limit the use of the library. I would like to avoid such a situation. |
Don't overthink this.
Option 2 is probably harder to maintain long term. Great project! I hope you add ts support soon. |
Yeah, my concern was about Svelte/SvelteKit projects that use the library's source code. But after some research, I realized that it is possible to get pure JS Svelte source files as a result of compiling Typescript source code. |
Any updates on this? |
Unfortunately, nothing was done on this task. |
I'm sorry.. the fullcalendar model with the options object ended up not working for me, so I lost interest. |
Is there an update on this? I am quite interested in using this. ts support is a necessity for my use case |
Are there plans to also support typescript?
The text was updated successfully, but these errors were encountered: