Skip to content
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

Ts updates #1625

Merged
merged 3 commits into from
May 14, 2019
Merged

Ts updates #1625

merged 3 commits into from
May 14, 2019

Conversation

tagyoureit
Copy link
Contributor

It is valid for a user to pass logger.info({some:object}) or the same logger.log('info',{some:object}) (or even an array or error) and unfortunately that means we need to add an any type to this declaration.

Added missing handleExceptions boolean in LoggerOptions
It is valid for a user to pass `logger.info({some:object})` or the same `logger.log('info',{some:object})` (or even an array or error) and unfortunately that means we need to add an any type to this declaration.
@indexzero indexzero requested a review from DABH May 14, 2019 02:54
Copy link
Contributor

@DABH DABH left a comment

Choose a reason for hiding this comment

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

LGTM

@DABH DABH merged commit 4e939b3 into winstonjs:master May 14, 2019
@tagyoureit tagyoureit deleted the TS_updates branch May 15, 2019 16:36
Mizumaki pushed a commit to Mizumaki/winston that referenced this pull request Jun 11, 2020
* Update index.d.ts

Added missing handleExceptions boolean in LoggerOptions

* Add :any to logmethod and loglevelmethod

It is valid for a user to pass `logger.info({some:object})` or the same `logger.log('info',{some:object})` (or even an array or error) and unfortunately that means we need to add an any type to this declaration.

* Added Logger return method to declarations

(forgot previously)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants