-
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
String enums have reverse mapping #16761
Comments
Can you share more of your configuration? I'm not seeing this C:\Throwaway>type q.ts
enum ActionType {
AddUser = "ADD_USER",
DeleteUser = "DELETE_USER",
RenameUser = "RENAME_USER"
}
C:\Throwaway>type q.js
var ActionType;
(function (ActionType) {
ActionType["AddUser"] = "ADD_USER";
ActionType["DeleteUser"] = "DELETE_USER";
ActionType["RenameUser"] = "RENAME_USER";
})(ActionType || (ActionType = {})); |
@RyanCavanaugh the question seems to be with respect to |
Reading fail 😢 |
This is what I see enum Mixed {
A,
B,
C = "hi",
D = 10,
E,
F = "bye"
} var Mixed;
(function (Mixed) {
Mixed[Mixed["A"] = 0] = "A";
Mixed[Mixed["B"] = 1] = "B";
Mixed["C"] = "hi";
Mixed[Mixed["D"] = 10] = "D";
Mixed[Mixed["E"] = 11] = "E";
Mixed["F"] = "bye";
})(Mixed || (Mixed = {})); |
@RyanCavanaugh My mistake not properly explaining - not just for mixed(used it as a ready example). When you asked for |
My mistake, the out-of-the-box build task used a different different version of TS. Don't know why the output console did not auto show. |
TypeScript Version: 2.4.0 / nightly (2.5.0-dev.20170627)
In the PR for string enums it is said they do not have reverse mapping. When I compile the following I get a different output from the shown in the PR. Is this a change?
Code
Expected behavior:
Actual behavior:
The text was updated successfully, but these errors were encountered: