-
Notifications
You must be signed in to change notification settings - Fork 11.9k
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
Remove I prefix from TS types #8017
Conversation
Something collides? |
Perhaps, but the TSC lint works. On the command line I see circular errors, but I think those are pre-existing.
|
I got the build working locally. I think we'll need to investigate the linter, some of these should have been caught by that. There were two bad imports ( |
Standardizes all of the typescript types to drop the
I
prefix. For exampleIEvent
is now justEvent
.The only non-standard change is
IInteractionMode
which changed toInteractionModeMap
becauseInteractionMode
was already defined.