-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Update typescript definitions to 3.0.0-* #1190
Comments
It's a work in progress, should be done soon... can follow the conversation here: |
@DABH would be fine adding them to the repository itself so they can be closely aligned on versioning. |
Now that I have access I'll bring the typings over here. I'll add the ones for 3.x; also, it might be nice (though not necessary) to bring the 2.x typings over as well, so everything's out of DefinitelyTyped. |
This would definitely be the preferred route |
Hey @DABH what's the status? |
@nolazybits I'm meeting with @indexzero on Friday and will discuss getting this integrated -- know this has been delayed for a while but expect some updates hopefully this weekend! |
logform PR is opened (winstonjs/logform#15); PRs for winston-transport and winston should come soon. Feel free to review/+1 as appropriate. |
|
|
#1287 Anyone who wants to help test/debug this scores some Internet points ;) Otherwise I'll test things as best I can and hopefully get this merged in soon. There may be problems with what's written, as well as missing things. I chose to exclude any deprecated methods, since people who are bleeding edge enough to use Typescript surely wouldn't want to use any deprecated methods ;) |
Typings have been merged and will be included in the next RC (if we do another one) and the final 3.0 release! Please start using them when you can and let me know if I missed anything. Thanks! |
Hi, thank you @DABH. I've just added the latest winston RC as dependency to my project and I'd be glad to use the TS definitions. Is a new RC expected before the final 3.0 or should I take definitions directly from github? |
@sveyret The TS defs should be in the latest RC (RC6). Let me know if you don’t see them! I don’t expect them to change significantly between now and the final 3.0 release near the end of the month. |
@DABH actually, I don't see this latest RC6 at all… Neither in github releases nor with npm. I tried to install winston@next, but I only have RC5, so I tried winston@3.0.0-rc6 but the version wasn't found. |
My apologies -- we are putting in a few more fixes before pushing RC6 (I suspect it will go out this Friday morning). Meanwhile if you use |
Thank you for the info. I think I'd better wait for Friday (morning for which continent? 😉) If it's not ready this week-end, I'll get the branch directly from git. |
Aiming for around pushing in around 11 hours but stay tuned ; ) |
A bit delayed? |
Indeed... @indexzero ? Perhaps just use |
Hi @DABH , |
I also noticed that there is a new |
I sent a pull request #1323 to correct this. I did not dare to add (un)handleExceptions because I don't use them and therefore am not sure of me. |
Thanks @sveyret for your contribution. We merged #1323 . I've opened #1330 which should address your comments on |
Thank you for merging this change. Your comment on |
TypeScript definitions for winston@3.0 (closes winstonjs#1190)
Not sure if this is maintained by you but the typescript definitions for RC don't yet exist ,
if this is maintained here could you bring these up to date
The text was updated successfully, but these errors were encountered: