Skip to content

TransformableInfo should support symbols as keys#240

Merged
wbt merged 2 commits intowinstonjs:masterfrom
dancrumb:patch-1
Feb 6, 2023
Merged

TransformableInfo should support symbols as keys#240
wbt merged 2 commits intowinstonjs:masterfrom
dancrumb:patch-1

Conversation

@dancrumb
Copy link
Copy Markdown
Contributor

Being able to use a Symbol as a key for TransformableInfo makes it easier to create cleaner transforms.

The current typing forbids this, so this change enables it.

Being able to use a Symbol as a key for TransformableInfo makes it easier to create cleaner transforms.

The current typing forbids this, so this change enables it.
@dscalzi
Copy link
Copy Markdown

dscalzi commented Feb 3, 2023

cc @wbt please merge this, it has been a problem for over two years. It's incredibly annoying to have to copy the same typings patch on every project that uses winston.

This project looks largely unmaintained, but this is a zero-effort solution to a frustrating issue.

@dscalzi
Copy link
Copy Markdown

dscalzi commented Feb 3, 2023

reference: #111 #114

@wbt
Copy link
Copy Markdown
Contributor

wbt commented Feb 7, 2023

cc @wbt please merge this, it has been a problem for over two years. It's incredibly annoying to have to copy the same typings patch on every project that uses winston.

FYI @dscalzi, that claim seems a bit less than truthful given that TS support for this is only about a year and a half old.

this is a zero-effort solution

Also not true, for me or according to the folks complaining at #244.

@dscalzi
Copy link
Copy Markdown

dscalzi commented Feb 8, 2023

The issue was first reported more than 2 years ago and never received a response. Moreover, this package has provided a .d.ts file for 5 years https://github.com/winstonjs/logform/commits/master/index.d.ts, not only a year and a half.

@dscalzi
Copy link
Copy Markdown

dscalzi commented Feb 8, 2023

See #15 ...

@wbt
Copy link
Copy Markdown
Contributor

wbt commented Feb 9, 2023

Support for using symbols as index types, which is what this specific PR is about, has only been released for a year and a half.
That is what's causing problems at #244 - it's too recent a feature in TS.

@wbt wbt mentioned this pull request Feb 20, 2023
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