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

Update index.d.ts #1729

Merged
merged 1 commit into from
Dec 2, 2019
Merged

Update index.d.ts #1729

merged 1 commit into from
Dec 2, 2019

Conversation

iffyio
Copy link
Contributor

@iffyio iffyio commented Nov 8, 2019

Hoist type defs from the logform module.

Currently there isn't a way to write correct type definitions for
when e.g creating a format since the types for such objects are
declared in logform.
This patch move logform type defs so that they can be used by
user code - For example by doing:

import { Logform } from 'winston'; const f: Logform.Format;

Hoist type defs from the logform module.

Currently there isn't a way to write correct type definitions for
when e.g creating a format since the types for such objects are
declared in `logform`.
This patch move logform type defs so that they can be used by
user code - For example by doing:

`import { Logform } from 'winston'; const f: Logform.Format;`
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.

Seems safe to me, thanks!

@DABH DABH merged commit 8944598 into winstonjs:master Dec 2, 2019
@jomel
Copy link

jomel commented Feb 17, 2020

Good to see this has been merged, but when can we expect a release to include this?! @DABH

@DABH
Copy link
Contributor

DABH commented Feb 17, 2020

Up to @indexzero ...

Mizumaki pushed a commit to Mizumaki/winston that referenced this pull request Jun 11, 2020
Hoist type defs from the logform module.

Currently there isn't a way to write correct type definitions for
when e.g creating a format since the types for such objects are
declared in `logform`.
This patch move logform type defs so that they can be used by
user code - For example by doing:

`import { Logform } from 'winston'; const f: Logform.Format;`
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.

3 participants