-
Notifications
You must be signed in to change notification settings - Fork 12.6k
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
Add support of spread enum member #40998
Conversation
@typescript-bot pack this - who knows, maybe it will work |
Well.... It's WIP yet. |
thanks man! really usefull this supports ... syntax
can I suggest to add a support of
also what happens in case of overlaping
will AdvEvents.Start === Basic1Events.Start ? |
You can see the comment and design notes |
@typescript-bot pack this. |
Hey @Kingwl, I've packed this into an installable tgz. You can install it for testing by referencing it in your
and then running There is also a playground for this build. |
@andrewbranch @DanielRosenwasser @sandersn I have some confuse. Enum can be merge from many positions. enum A {
f = 'f'
}
console.log(A.ff) // [LOG]: undefined
enum B {
...A,
b = 'b'
}
enum A {
ff = 'ff'
}
console.log(A.f, A.ff, B.f, B.ff, B.b)
console.log(A.f, A.ff, B.f, B.ff, B.b) // [LOG]: "f", "ff", "f", undefined, "b" The enum I don't have background about that. So is that expected behavior? If not. I'd like to propose disallow spread if any enum declaration below of the current one. |
Well. It's seems incorrect implements. It's maybe should be handled as |
Passed all tests in my pc so. @typescript-bot pack this. |
Heya @Kingwl, I've started to run the perf test suite on this PR at ab6bcfc. You can monitor the build here. Update: The results are in! |
Hey @Kingwl, I've packed this into an installable tgz. You can install it for testing by referencing it in your
and then running There is also a playground for this build. |
@Kingwl Here they are:Comparison Report - master..40998
System
Hosts
Scenarios
|
seems basicly works. It's time to add some additional check and improvement the perf. |
@typescript-bot pack this. |
@typescript-bot perf test. |
Heya @Kingwl, I've started to run the perf test suite on this PR at 1b4935c. You can monitor the build here. Update: The results are in! |
Hey @Kingwl, I've packed this into an installable tgz. You can install it for testing by referencing it in your
and then running There is also a playground for this build. |
@Kingwl Here they are:Comparison Report - master..40998
System
Hosts
Scenarios
|
@typescript-bot perf test. |
Heya @Kingwl, I've started to run the perf test suite on this PR at 1b4935c. You can monitor the build here. Update: The results are in! |
@Kingwl Here they are:Comparison Report - master..40998
System
Hosts
Scenarios
|
@typescript-bot pack this. |
Heya @Kingwl, I've started to run the perf test suite on this PR at 4aa1fc3. You can monitor the build here. Update: The results are in! |
Hey @Kingwl, I've packed this into an installable tgz. You can install it for testing by referencing it in your
and then running There is also a playground for this build. |
@Kingwl Here they are:Comparison Report - master..40998
System
Hosts
Scenarios
|
Hi, any plans on merging this PR soon? |
Nope. |
This experiment is pretty old, so I'm going to close it to reduce the number of open PRs. |
The TypeScript team hasn't accepted the linked issue #17592. If you can get it accepted, this PR will have a better chance of being reviewed. |
The problem is still relevant |
Fixes #17592