-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
[milestone] Convert the library to typescript #4700
Comments
@martijnrusschen can you pin the issue? |
Nice progress so far! |
@mirus-ua ( @martijnrusschen ) |
refs #4000 |
@yuki0410-dev Feel free to claim a file and let it know here so there's no duplicate work done. |
@yuki0410-dev hello |
I would like to take the portal jsx file in work |
I would like to take the week_number.jsx file in work |
Added your name to week_number.jsx |
@mirus-ua |
@mirus-ua |
@martijnrusschen hello. Please ask reviews to look at four opened PRs if you can. It'll be much faster if we merge them because the rest of the modules depend on them |
Yep, I'll expect them to complete this soon. Normal turnaround time is ~2-4 hours. |
@mirus-ua |
@mirus-ua |
sure, I think you should wait a bit for the month and the year modules |
I agree with waiting too. |
apologies, I'm on vacation this week and don't have the ability to review
or merge anything until Friday.
…On Mon, Apr 29, 2024 at 3:15 PM yuki0410 ***@***.***> wrote:
@mirus-ua <https://github.com/mirus-ua>
sure, I think you should wait a bit for the month and the year modules
I agree with waiting too.
We are moving forward on the calendar.jsx work, but there are some aspects
that we cannot move forward on until those are completed.
—
Reply to this email directly, view it on GitHub
<#4700 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAKY2KE7BIGCPH4IGIOERD3Y7ZBWDAVCNFSM6AAAAABGK5XR6KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOBSG4YTINRRGA>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
@yuki0410-dev almost everything was merged, so you can start with the final part |
what's left? |
@mirus-ua (@martijnrusschen ) |
The core features are done.But I think better to migrate some amount of tests just to double check that all types are OK |
@mirus-ua |
everything is correct |
Contribution is welcome
Why?
Everything started here from the willing to merge a separate package with the corresponding type definitions into this repo.
The idea is great, but we, as a community, can go further and migrate the entire code base into typescript.
Any benefits?
Our steps to achieve the goal
.ts
and.tsx
modules [typescript-migration] initial ts config and the first calendar_container.jsx module #4704ts-jest
)Modules to migrate and how to contribute
Basic rules and advice:
calendar_container.jsx
any
or other loose types if you don't have strong argumentsSteps to contribute
[typescript-migration] calendar.jsx
etc.Breaking changes so far:
newDate
helper always returns a date, if the initial params were invalid, then it returns just anew Date
Line; PRThe text was updated successfully, but these errors were encountered: