Mark static ID properties as readonly

src/vs/editor/contrib/hover/modesContentHover.ts

@@ -154,7 +154,7 @@ class ModesContentComputer implements IHoverComputer<HoverPart[]> {
 
 export class ModesContentHoverWidget extends ContentHoverWidget {
 
-	static ID = 'editor.contrib.modesContentHoverWidget';
+	static readonly ID = 'editor.contrib.modesContentHoverWidget';
 
 	private _messages: HoverPart[];
 	private _lastRange: Range;

src/vs/editor/contrib/quickFix/quickFixCommands.ts

@@ -125,7 +125,7 @@ export class QuickFixController implements IEditorContribution {
 
 export class QuickFixAction extends EditorAction {
 
-	static Id = 'editor.action.quickFix';
+	static readonly Id = 'editor.action.quickFix';
 
 	constructor() {
 		super({

src/vs/workbench/browser/actions/workspaceActions.ts

@@ -30,7 +30,7 @@ import { IHistoryService } from 'vs/workbench/services/history/common/history';
 
 export class OpenFileAction extends Action {
 
-	static ID = 'workbench.action.files.openFile';
+	static readonly ID = 'workbench.action.files.openFile';
 	static LABEL = nls.localize('openFile', "Open File...");
 
 	constructor(
@@ -50,7 +50,7 @@ export class OpenFileAction extends Action {
 
 export class OpenFolderAction extends Action {
 
-	static ID = 'workbench.action.files.openFolder';
+	static readonly ID = 'workbench.action.files.openFolder';
 	static LABEL = nls.localize('openFolder', "Open Folder...");
 
 	constructor(
@@ -70,7 +70,7 @@ export class OpenFolderAction extends Action {
 
 export class OpenFileFolderAction extends Action {
 
-	static ID = 'workbench.action.files.openFileFolder';
+	static readonly ID = 'workbench.action.files.openFileFolder';
 	static LABEL = nls.localize('openFileFolder', "Open...");
 
 	constructor(
@@ -195,7 +195,7 @@ function isUntitledWorkspace(path: string, environmentService: IEnvironmentServi
 
 export class AddRootFolderAction extends BaseWorkspacesAction {
 
-	static ID = 'workbench.action.addRootFolder';
+	static readonly ID = 'workbench.action.addRootFolder';
 	static LABEL = nls.localize('addFolderToWorkspace', "Add Folder to Workspace...");
 
 	constructor(
@@ -225,7 +225,7 @@ export class AddRootFolderAction extends BaseWorkspacesAction {
 
 export class GlobalRemoveRootFolderAction extends BaseWorkspacesAction {
 
-	static ID = 'workbench.action.removeRootFolder';
+	static readonly ID = 'workbench.action.removeRootFolder';
 	static LABEL = nls.localize('globalRemoveFolderFromWorkspace', "Remove Folder from Workspace...");
 
 	constructor(
@@ -261,7 +261,7 @@ export class GlobalRemoveRootFolderAction extends BaseWorkspacesAction {
 
 export class RemoveRootFolderAction extends Action {
 
-	static ID = 'workbench.action.removeRootFolder';
+	static readonly ID = 'workbench.action.removeRootFolder';
 	static LABEL = nls.localize('removeFolderFromWorkspace', "Remove Folder from Workspace");
 
 	constructor(
@@ -280,7 +280,7 @@ export class RemoveRootFolderAction extends Action {
 
 export class OpenFolderSettingsAction extends Action {
 
-	static ID = 'workbench.action.openFolderSettings';
+	static readonly ID = 'workbench.action.openFolderSettings';
 	static LABEL = nls.localize('openFolderSettings', "Open Folder Settings");
 
 	constructor(
@@ -302,7 +302,7 @@ export class OpenFolderSettingsAction extends Action {
 
 export class SaveWorkspaceAsAction extends BaseWorkspacesAction {
 
-	static ID = 'workbench.action.saveWorkspaceAs';
+	static readonly ID = 'workbench.action.saveWorkspaceAs';
 	static LABEL = nls.localize('saveWorkspaceAsAction', "Save Workspace As...");
 
 	constructor(
@@ -347,7 +347,7 @@ export class SaveWorkspaceAsAction extends BaseWorkspacesAction {
 
 export class OpenWorkspaceAction extends Action {
 
-	static ID = 'workbench.action.openWorkspace';
+	static readonly ID = 'workbench.action.openWorkspace';
 	static LABEL = nls.localize('openWorkspaceAction', "Open Workspace...");
 
 	constructor(

src/vs/workbench/browser/parts/editor/editorStatus.ts

@@ -765,7 +765,7 @@ function isWritableBaseEditor(e: IBaseEditor): boolean {
 
 export class ShowLanguageExtensionsAction extends Action {
 
-	static ID = 'workbench.action.showLanguageExtensions';
+	static readonly ID = 'workbench.action.showLanguageExtensions';
 
 	constructor(
 		private fileExtension: string,

src/vs/workbench/browser/parts/panel/panelActions.ts

@@ -18,7 +18,7 @@ import { ActivityAction } from 'vs/workbench/browser/parts/compositebar/composit
 import { IActivity } from 'vs/workbench/common/activity';
 
 export class ClosePanelAction extends Action {
-	static ID = 'workbench.action.closePanel';
+	static readonly ID = 'workbench.action.closePanel';
 	static LABEL = nls.localize('closePanel', "Close Panel");
 
 	constructor(
@@ -35,7 +35,7 @@ export class ClosePanelAction extends Action {
 }
 
 export class TogglePanelAction extends Action {
-	static ID = 'workbench.action.togglePanel';
+	static readonly ID = 'workbench.action.togglePanel';
 	static LABEL = nls.localize('togglePanel', "Toggle Panel");
 
 	constructor(

src/vs/workbench/electron-browser/actions.ts

@@ -91,7 +91,7 @@ export class CloseCurrentWindowAction extends Action {
 
 export class CloseWorkspaceAction extends Action {
 
-	static ID = 'workbench.action.closeFolder';
+	static readonly ID = 'workbench.action.closeFolder';
 	static LABEL = nls.localize('closeWorkspace', "Close Workspace");
 
 	constructor(
@@ -117,7 +117,7 @@ export class CloseWorkspaceAction extends Action {
 
 export class NewWindowAction extends Action {
 
-	static ID = 'workbench.action.newWindow';
+	static readonly ID = 'workbench.action.newWindow';
 	static LABEL = nls.localize('newWindow', "New Window");
 
 	constructor(
@@ -135,7 +135,7 @@ export class NewWindowAction extends Action {
 
 export class ToggleFullScreenAction extends Action {
 
-	static ID = 'workbench.action.toggleFullScreen';
+	static readonly ID = 'workbench.action.toggleFullScreen';
 	static LABEL = nls.localize('toggleFullScreen', "Toggle Full Screen");
 
 	constructor(id: string, label: string, @IWindowService private windowService: IWindowService) {
@@ -149,7 +149,7 @@ export class ToggleFullScreenAction extends Action {
 
 export class ToggleMenuBarAction extends Action {
 
-	static ID = 'workbench.action.toggleMenuBar';
+	static readonly ID = 'workbench.action.toggleMenuBar';
 	static LABEL = nls.localize('toggleMenuBar', "Toggle Menu Bar");
 
 	private static readonly menuBarVisibilityKey = 'window.menuBarVisibility';
@@ -183,7 +183,7 @@ export class ToggleMenuBarAction extends Action {
 
 export class ToggleDevToolsAction extends Action {
 
-	static ID = 'workbench.action.toggleDevTools';
+	static readonly ID = 'workbench.action.toggleDevTools';
 	static LABEL = nls.localize('toggleDevTools', "Toggle Developer Tools");
 
 	constructor(id: string, label: string, @IWindowService private windowsService: IWindowService) {
@@ -558,7 +558,7 @@ export class ShowStartupPerformance extends Action {
 
 export class ReloadWindowAction extends Action {
 
-	static ID = 'workbench.action.reloadWindow';
+	static readonly ID = 'workbench.action.reloadWindow';
 	static LABEL = nls.localize('reloadWindow', "Reload Window");
 
 	constructor(
@@ -654,7 +654,7 @@ class CloseWindowAction extends Action implements IPickOpenAction {
 
 export class SwitchWindow extends BaseSwitchWindow {
 
-	static ID = 'workbench.action.switchWindow';
+	static readonly ID = 'workbench.action.switchWindow';
 	static LABEL = nls.localize('switchWindow', "Switch Window...");
 
 	constructor(
@@ -676,7 +676,7 @@ export class SwitchWindow extends BaseSwitchWindow {
 
 export class QuickSwitchWindow extends BaseSwitchWindow {
 
-	static ID = 'workbench.action.quickSwitchWindow';
+	static readonly ID = 'workbench.action.quickSwitchWindow';
 	static LABEL = nls.localize('quickSwitchWindow', "Quick Switch Window...");
 
 	constructor(
@@ -1189,7 +1189,7 @@ export class OpenTipsAndTricksUrlAction extends Action {
 
 export class ToggleSharedProcessAction extends Action {
 
-	static ID = 'workbench.action.toggleSharedProcess';
+	static readonly ID = 'workbench.action.toggleSharedProcess';
 	static LABEL = nls.localize('toggleSharedProcess', "Toggle Shared Process");
 
 	constructor(id: string, label: string, @IWindowsService private windowsService: IWindowsService) {
@@ -1687,7 +1687,7 @@ export class ConfigureLocaleAction extends Action {
 
 export class OpenLogsFolderAction extends Action {
 
-	static ID = 'workbench.action.openLogsFolder';
+	static readonly ID = 'workbench.action.openLogsFolder';
 	static LABEL = nls.localize('openLogsFolder', "Open Logs Folder");
 
 	constructor(id: string, label: string,
@@ -1704,7 +1704,7 @@ export class OpenLogsFolderAction extends Action {
 
 export class ShowLogsAction extends Action {
 
-	static ID = 'workbench.action.showLogs';
+	static readonly ID = 'workbench.action.showLogs';
 	static LABEL = nls.localize('showLogs', "Show Logs...");
 
 	constructor(id: string, label: string,
@@ -1734,7 +1734,7 @@ export class ShowLogsAction extends Action {
 
 export class SetLogLevelAction extends Action {
 
-	static ID = 'workbench.action.setLogLevel';
+	static readonly ID = 'workbench.action.setLogLevel';
 	static LABEL = nls.localize('setLogLevel', "Set Log Level");
 
 	constructor(id: string, label: string,

src/vs/workbench/parts/cli/electron-browser/cli.contribution.ts

@@ -30,7 +30,7 @@ function isAvailable(): TPromise<boolean> {
 
 class InstallAction extends Action {
 
-	static ID = 'workbench.action.installCommandLine';
+	static readonly ID = 'workbench.action.installCommandLine';
 	static LABEL = nls.localize('install', "Install '{0}' command in PATH", product.applicationName);
 
 	constructor(
@@ -111,7 +111,7 @@ class InstallAction extends Action {
 
 class UninstallAction extends Action {
 
-	static ID = 'workbench.action.uninstallCommandLine';
+	static readonly ID = 'workbench.action.uninstallCommandLine';
 	static LABEL = nls.localize('uninstall', "Uninstall '{0}' command from PATH", product.applicationName);
 
 	constructor(

src/vs/workbench/parts/debug/browser/debugActions.ts

@@ -71,7 +71,7 @@ export abstract class AbstractDebugAction extends Action {
 }
 
 export class ConfigureAction extends AbstractDebugAction {
-	static ID = 'workbench.action.debug.configure';
+	static readonly ID = 'workbench.action.debug.configure';
 	static LABEL = nls.localize('openLaunchJson', "Open {0}", 'launch.json');
 
 	constructor(id: string, label: string,
@@ -160,7 +160,7 @@ export class StartAction extends AbstractDebugAction {
 }
 
 export class RunAction extends StartAction {
-	static ID = 'workbench.action.debug.run';
+	static readonly ID = 'workbench.action.debug.run';
 	static LABEL = nls.localize('startWithoutDebugging', "Start Without Debugging");
 
 	protected isNoDebug(): boolean {
@@ -169,7 +169,7 @@ export class RunAction extends StartAction {
 }
 
 export class SelectAndStartAction extends AbstractDebugAction {
-	static ID = 'workbench.action.debug.selectandstart';
+	static readonly ID = 'workbench.action.debug.selectandstart';
 	static LABEL = nls.localize('selectAndStartDebugging', "Select and Start Debugging");
 
 	constructor(id: string, label: string,
@@ -190,7 +190,7 @@ export class SelectAndStartAction extends AbstractDebugAction {
 }
 
 export class RestartAction extends AbstractDebugAction {
-	static ID = 'workbench.action.debug.restart';
+	static readonly ID = 'workbench.action.debug.restart';
 	static LABEL = nls.localize('restartDebug', "Restart");
 	static RECONNECT_LABEL = nls.localize('reconnectDebug', "Reconnect");
 
@@ -224,7 +224,7 @@ export class RestartAction extends AbstractDebugAction {
 }
 
 export class StepOverAction extends AbstractDebugAction {
-	static ID = 'workbench.action.debug.stepOver';
+	static readonly ID = 'workbench.action.debug.stepOver';
 	static LABEL = nls.localize('stepOverDebug', "Step Over");
 
 	constructor(id: string, label: string, @IDebugService debugService: IDebugService, @IKeybindingService keybindingService: IKeybindingService) {
@@ -245,7 +245,7 @@ export class StepOverAction extends AbstractDebugAction {
 }
 
 export class StepIntoAction extends AbstractDebugAction {
-	static ID = 'workbench.action.debug.stepInto';
+	static readonly ID = 'workbench.action.debug.stepInto';
 	static LABEL = nls.localize('stepIntoDebug', "Step Into");
 
 	constructor(id: string, label: string, @IDebugService debugService: IDebugService, @IKeybindingService keybindingService: IKeybindingService) {
@@ -266,7 +266,7 @@ export class StepIntoAction extends AbstractDebugAction {
 }
 
 export class StepOutAction extends AbstractDebugAction {
-	static ID = 'workbench.action.debug.stepOut';
+	static readonly ID = 'workbench.action.debug.stepOut';
 	static LABEL = nls.localize('stepOutDebug', "Step Out");
 
 	constructor(id: string, label: string, @IDebugService debugService: IDebugService, @IKeybindingService keybindingService: IKeybindingService) {
@@ -287,7 +287,7 @@ export class StepOutAction extends AbstractDebugAction {
 }
 
 export class StopAction extends AbstractDebugAction {
-	static ID = 'workbench.action.debug.stop';
+	static readonly ID = 'workbench.action.debug.stop';
 	static LABEL = nls.localize('stopDebug', "Stop");
 
 	constructor(id: string, label: string, @IDebugService debugService: IDebugService, @IKeybindingService keybindingService: IKeybindingService) {
@@ -308,7 +308,7 @@ export class StopAction extends AbstractDebugAction {
 }
 
 export class DisconnectAction extends AbstractDebugAction {
-	static ID = 'workbench.action.debug.disconnect';
+	static readonly ID = 'workbench.action.debug.disconnect';
 	static LABEL = nls.localize('disconnectDebug', "Disconnect");
 
 	constructor(id: string, label: string, @IDebugService debugService: IDebugService, @IKeybindingService keybindingService: IKeybindingService) {
@@ -326,7 +326,7 @@ export class DisconnectAction extends AbstractDebugAction {
 }
 
 export class ContinueAction extends AbstractDebugAction {
-	static ID = 'workbench.action.debug.continue';
+	static readonly ID = 'workbench.action.debug.continue';
 	static LABEL = nls.localize('continueDebug', "Continue");
 
 	constructor(id: string, label: string, @IDebugService debugService: IDebugService, @IKeybindingService keybindingService: IKeybindingService) {
@@ -347,7 +347,7 @@ export class ContinueAction extends AbstractDebugAction {
 }
 
 export class PauseAction extends AbstractDebugAction {
-	static ID = 'workbench.action.debug.pause';
+	static readonly ID = 'workbench.action.debug.pause';
 	static LABEL = nls.localize('pauseDebug', "Pause");
 
 	constructor(id: string, label: string, @IDebugService debugService: IDebugService, @IKeybindingService keybindingService: IKeybindingService) {
@@ -368,7 +368,7 @@ export class PauseAction extends AbstractDebugAction {
 }
 
 export class RestartFrameAction extends AbstractDebugAction {
-	static ID = 'workbench.action.debug.restartFrame';
+	static readonly ID = 'workbench.action.debug.restartFrame';
 	static LABEL = nls.localize('restartFrame', "Restart Frame");
 
 	constructor(id: string, label: string, @IDebugService debugService: IDebugService, @IKeybindingService keybindingService: IKeybindingService) {
@@ -385,7 +385,7 @@ export class RestartFrameAction extends AbstractDebugAction {
 }
 
 export class RemoveBreakpointAction extends AbstractDebugAction {
-	static ID = 'workbench.debug.viewlet.action.removeBreakpoint';
+	static readonly ID = 'workbench.debug.viewlet.action.removeBreakpoint';
 	static LABEL = nls.localize('removeBreakpoint', "Remove Breakpoint");
 
 	constructor(id: string, label: string, @IDebugService debugService: IDebugService, @IKeybindingService keybindingService: IKeybindingService) {
@@ -399,7 +399,7 @@ export class RemoveBreakpointAction extends AbstractDebugAction {
 }
 
 export class RemoveAllBreakpointsAction extends AbstractDebugAction {
-	static ID = 'workbench.debug.viewlet.action.removeAllBreakpoints';
+	static readonly ID = 'workbench.debug.viewlet.action.removeAllBreakpoints';
 	static LABEL = nls.localize('removeAllBreakpoints', "Remove All Breakpoints");
 
 	constructor(id: string, label: string, @IDebugService debugService: IDebugService, @IKeybindingService keybindingService: IKeybindingService) {
@@ -418,7 +418,7 @@ export class RemoveAllBreakpointsAction extends AbstractDebugAction {
 }
 
 export class EnableBreakpointAction extends AbstractDebugAction {
-	static ID = 'workbench.debug.viewlet.action.enableBreakpoint';
+	static readonly ID = 'workbench.debug.viewlet.action.enableBreakpoint';
 	static LABEL = nls.localize('enableBreakpoint', "Enable Breakpoint");
 
 	constructor(id: string, label: string, @IDebugService debugService: IDebugService, @IKeybindingService keybindingService: IKeybindingService) {
@@ -431,7 +431,7 @@ export class EnableBreakpointAction extends AbstractDebugAction {
 }
 
 export class DisableBreakpointAction extends AbstractDebugAction {
-	static ID = 'workbench.debug.viewlet.action.disableBreakpoint';
+	static readonly ID = 'workbench.debug.viewlet.action.disableBreakpoint';
 	static LABEL = nls.localize('disableBreakpoint', "Disable Breakpoint");
 
 	constructor(id: string, label: string, @IDebugService debugService: IDebugService, @IKeybindingService keybindingService: IKeybindingService) {
@@ -444,7 +444,7 @@ export class DisableBreakpointAction extends AbstractDebugAction {
 }
 
 export class EnableAllBreakpointsAction extends AbstractDebugAction {
-	static ID = 'workbench.debug.viewlet.action.enableAllBreakpoints';
+	static readonly ID = 'workbench.debug.viewlet.action.enableAllBreakpoints';
 	static LABEL = nls.localize('enableAllBreakpoints', "Enable All Breakpoints");
 
 	constructor(id: string, label: string, @IDebugService debugService: IDebugService, @IKeybindingService keybindingService: IKeybindingService) {
@@ -463,7 +463,7 @@ export class EnableAllBreakpointsAction extends AbstractDebugAction {
 }
 
 export class DisableAllBreakpointsAction extends AbstractDebugAction {
-	static ID = 'workbench.debug.viewlet.action.disableAllBreakpoints';
+	static readonly ID = 'workbench.debug.viewlet.action.disableAllBreakpoints';
 	static LABEL = nls.localize('disableAllBreakpoints', "Disable All Breakpoints");
 
 	constructor(id: string, label: string, @IDebugService debugService: IDebugService, @IKeybindingService keybindingService: IKeybindingService) {
@@ -482,7 +482,7 @@ export class DisableAllBreakpointsAction extends AbstractDebugAction {
 }
 
 export class ToggleBreakpointsActivatedAction extends AbstractDebugAction {
-	static ID = 'workbench.debug.viewlet.action.toggleBreakpointsActivatedAction';
+	static readonly ID = 'workbench.debug.viewlet.action.toggleBreakpointsActivatedAction';
 	static ACTIVATE_LABEL = nls.localize('activateBreakpoints', "Activate Breakpoints");
 	static DEACTIVATE_LABEL = nls.localize('deactivateBreakpoints', "Deactivate Breakpoints");
 
@@ -506,7 +506,7 @@ export class ToggleBreakpointsActivatedAction extends AbstractDebugAction {
 }
 
 export class ReapplyBreakpointsAction extends AbstractDebugAction {
-	static ID = 'workbench.debug.viewlet.action.reapplyBreakpointsAction';
+	static readonly ID = 'workbench.debug.viewlet.action.reapplyBreakpointsAction';
 	static LABEL = nls.localize('reapplyAllBreakpoints', "Reapply All Breakpoints");
 
 	constructor(id: string, label: string, @IDebugService debugService: IDebugService, @IKeybindingService keybindingService: IKeybindingService) {
@@ -526,7 +526,7 @@ export class ReapplyBreakpointsAction extends AbstractDebugAction {
 }
 
 export class AddFunctionBreakpointAction extends AbstractDebugAction {
-	static ID = 'workbench.debug.viewlet.action.addFunctionBreakpointAction';
+	static readonly ID = 'workbench.debug.viewlet.action.addFunctionBreakpointAction';
 	static LABEL = nls.localize('addFunctionBreakpoint', "Add Function Breakpoint");
 
 	constructor(id: string, label: string, @IDebugService debugService: IDebugService, @IKeybindingService keybindingService: IKeybindingService) {
@@ -546,7 +546,7 @@ export class AddFunctionBreakpointAction extends AbstractDebugAction {
 }
 
 export class AddConditionalBreakpointAction extends AbstractDebugAction {
-	static ID = 'workbench.debug.viewlet.action.addConditionalBreakpointAction';
+	static readonly ID = 'workbench.debug.viewlet.action.addConditionalBreakpointAction';
 	static LABEL = nls.localize('addConditionalBreakpoint', "Add Conditional Breakpoint...");
 
 	constructor(id: string, label: string,
@@ -565,7 +565,7 @@ export class AddConditionalBreakpointAction extends AbstractDebugAction {
 }
 
 export class EditConditionalBreakpointAction extends AbstractDebugAction {
-	static ID = 'workbench.debug.viewlet.action.editConditionalBreakpointAction';
+	static readonly ID = 'workbench.debug.viewlet.action.editConditionalBreakpointAction';
 	static LABEL = nls.localize('editConditionalBreakpoint', "Edit Breakpoint...");
 
 	constructor(id: string, label: string,
@@ -584,7 +584,7 @@ export class EditConditionalBreakpointAction extends AbstractDebugAction {
 
 
 export class SetValueAction extends AbstractDebugAction {
-	static ID = 'workbench.debug.viewlet.action.setValue';
+	static readonly ID = 'workbench.debug.viewlet.action.setValue';
 	static LABEL = nls.localize('setValue', "Set Value");
 
 	constructor(id: string, label: string, private variable: Variable, @IDebugService debugService: IDebugService, @IKeybindingService keybindingService: IKeybindingService) {
@@ -607,7 +607,7 @@ export class SetValueAction extends AbstractDebugAction {
 
 
 export class AddWatchExpressionAction extends AbstractDebugAction {
-	static ID = 'workbench.debug.viewlet.action.addWatchExpression';
+	static readonly ID = 'workbench.debug.viewlet.action.addWatchExpression';
 	static LABEL = nls.localize('addWatchExpression', "Add Expression");
 
 	constructor(id: string, label: string, @IDebugService debugService: IDebugService, @IKeybindingService keybindingService: IKeybindingService) {
@@ -626,7 +626,7 @@ export class AddWatchExpressionAction extends AbstractDebugAction {
 }
 
 export class EditWatchExpressionAction extends AbstractDebugAction {
-	static ID = 'workbench.debug.viewlet.action.editWatchExpression';
+	static readonly ID = 'workbench.debug.viewlet.action.editWatchExpression';
 	static LABEL = nls.localize('editWatchExpression', "Edit Expression");
 
 	constructor(id: string, label: string, @IDebugService debugService: IDebugService, @IKeybindingService keybindingService: IKeybindingService) {
@@ -640,7 +640,7 @@ export class EditWatchExpressionAction extends AbstractDebugAction {
 }
 
 export class AddToWatchExpressionsAction extends AbstractDebugAction {
-	static ID = 'workbench.debug.viewlet.action.addToWatchExpressions';
+	static readonly ID = 'workbench.debug.viewlet.action.addToWatchExpressions';
 	static LABEL = nls.localize('addToWatchExpressions', "Add to Watch");
 
 	constructor(id: string, label: string, private expression: IExpression, @IDebugService debugService: IDebugService, @IKeybindingService keybindingService: IKeybindingService) {
@@ -656,7 +656,7 @@ export class AddToWatchExpressionsAction extends AbstractDebugAction {
 }
 
 export class RemoveWatchExpressionAction extends AbstractDebugAction {
-	static ID = 'workbench.debug.viewlet.action.removeWatchExpression';
+	static readonly ID = 'workbench.debug.viewlet.action.removeWatchExpression';
 	static LABEL = nls.localize('removeWatchExpression', "Remove Expression");
 
 	constructor(id: string, label: string, @IDebugService debugService: IDebugService, @IKeybindingService keybindingService: IKeybindingService) {
@@ -670,7 +670,7 @@ export class RemoveWatchExpressionAction extends AbstractDebugAction {
 }
 
 export class RemoveAllWatchExpressionsAction extends AbstractDebugAction {
-	static ID = 'workbench.debug.viewlet.action.removeAllWatchExpressions';
+	static readonly ID = 'workbench.debug.viewlet.action.removeAllWatchExpressions';
 	static LABEL = nls.localize('removeAllWatchExpressions', "Remove All Expressions");
 
 	constructor(id: string, label: string, @IDebugService debugService: IDebugService, @IKeybindingService keybindingService: IKeybindingService) {
@@ -689,7 +689,7 @@ export class RemoveAllWatchExpressionsAction extends AbstractDebugAction {
 }
 
 export class ClearReplAction extends AbstractDebugAction {
-	static ID = 'workbench.debug.panel.action.clearReplAction';
+	static readonly ID = 'workbench.debug.panel.action.clearReplAction';
 	static LABEL = nls.localize('clearRepl', "Clear Console");
 
 	constructor(id: string, label: string,
@@ -709,7 +709,7 @@ export class ClearReplAction extends AbstractDebugAction {
 }
 
 export class ToggleReplAction extends TogglePanelAction {
-	static ID = 'workbench.debug.action.toggleRepl';
+	static readonly ID = 'workbench.debug.action.toggleRepl';
 	static LABEL = nls.localize({ comment: ['Debug is a noun in this context, not a verb.'], key: 'debugConsoleAction' }, 'Debug Console');
 	private toDispose: lifecycle.IDisposable[];
 
@@ -751,7 +751,7 @@ export class ToggleReplAction extends TogglePanelAction {
 
 export class FocusReplAction extends Action {
 
-	static ID = 'workbench.debug.action.focusRepl';
+	static readonly ID = 'workbench.debug.action.focusRepl';
 	static LABEL = nls.localize({ comment: ['Debug is a noun in this context, not a verb.'], key: 'debugFocusConsole' }, 'Focus Debug Console');
 
 
@@ -767,7 +767,7 @@ export class FocusReplAction extends Action {
 }
 
 export class FocusProcessAction extends AbstractDebugAction {
-	static ID = 'workbench.action.debug.focusProcess';
+	static readonly ID = 'workbench.action.debug.focusProcess';
 	static LABEL = nls.localize('focusProcess', "Focus Process");
 
 	constructor(id: string, label: string,
@@ -793,7 +793,7 @@ export class FocusProcessAction extends AbstractDebugAction {
 
 // Actions used by the chakra debugger
 export class StepBackAction extends AbstractDebugAction {
-	static ID = 'workbench.action.debug.stepBack';
+	static readonly ID = 'workbench.action.debug.stepBack';
 	static LABEL = nls.localize('stepBackDebug', "Step Back");
 
 	constructor(id: string, label: string, @IDebugService debugService: IDebugService, @IKeybindingService keybindingService: IKeybindingService) {
@@ -816,7 +816,7 @@ export class StepBackAction extends AbstractDebugAction {
 }
 
 export class ReverseContinueAction extends AbstractDebugAction {
-	static ID = 'workbench.action.debug.reverseContinue';
+	static readonly ID = 'workbench.action.debug.reverseContinue';
 	static LABEL = nls.localize('reverseContinue', "Reverse");
 
 	constructor(id: string, label: string, @IDebugService debugService: IDebugService, @IKeybindingService keybindingService: IKeybindingService) {

src/vs/workbench/parts/debug/browser/debugEditorModelManager.ts

@@ -32,7 +32,7 @@ interface IDebugEditorModelData {
 const stickiness = TrackedRangeStickiness.NeverGrowsWhenTypingAtEdges;
 
 export class DebugEditorModelManager implements IWorkbenchContribution {
-	static ID = 'breakpointManager';
+	static readonly ID = 'breakpointManager';
 
 	private modelDataMap: Map<string, IDebugEditorModelData>;
 	private toDispose: lifecycle.IDisposable[];

src/vs/workbench/parts/debug/browser/debugViewlet.ts

@@ -138,7 +138,7 @@ export class DebugViewlet extends PersistentViewsViewlet {
 
 export class FocusVariablesViewAction extends Action {
 
-	static ID = 'workbench.debug.action.focusVariablesView';
+	static readonly ID = 'workbench.debug.action.focusVariablesView';
 	static LABEL = nls.localize('debugFocusVariablesView', 'Focus Variables');
 
 	constructor(id: string, label: string,
@@ -156,7 +156,7 @@ export class FocusVariablesViewAction extends Action {
 
 export class FocusWatchViewAction extends Action {
 
-	static ID = 'workbench.debug.action.focusWatchView';
+	static readonly ID = 'workbench.debug.action.focusWatchView';
 	static LABEL = nls.localize({ comment: ['Debug is a noun in this context, not a verb.'], key: 'debugFocusWatchView' }, 'Focus Watch');
 
 	constructor(id: string, label: string,
@@ -174,7 +174,7 @@ export class FocusWatchViewAction extends Action {
 
 export class FocusCallStackViewAction extends Action {
 
-	static ID = 'workbench.debug.action.focusCallStackView';
+	static readonly ID = 'workbench.debug.action.focusCallStackView';
 	static LABEL = nls.localize({ comment: ['Debug is a noun in this context, not a verb.'], key: 'debugFocusCallStackView' }, 'Focus CallStack');
 
 	constructor(id: string, label: string,
@@ -192,7 +192,7 @@ export class FocusCallStackViewAction extends Action {
 
 export class FocusBreakpointsViewAction extends Action {
 
-	static ID = 'workbench.debug.action.focusBreakpointsView';
+	static readonly ID = 'workbench.debug.action.focusBreakpointsView';
 	static LABEL = nls.localize({ comment: ['Debug is a noun in this context, not a verb.'], key: 'debugFocusBreakpointsView' }, 'Focus Breakpoints');
 
 	constructor(id: string, label: string,

src/vs/workbench/parts/debug/electron-browser/breakpointsView.ts

@@ -267,7 +267,7 @@ class BreakpointsRenderer implements IRenderer<IBreakpoint, IBreakpointTemplateD
 		// noop
 	}
 
-	static ID = 'breakpoints';
+	static readonly ID = 'breakpoints';
 
 	get templateId() {
 		return BreakpointsRenderer.ID;
@@ -331,7 +331,7 @@ class ExceptionBreakpointsRenderer implements IRenderer<IExceptionBreakpoint, IB
 		// noop
 	}
 
-	static ID = 'exceptionbreakpoints';
+	static readonly ID = 'exceptionbreakpoints';
 
 	get templateId() {
 		return ExceptionBreakpointsRenderer.ID;
@@ -376,7 +376,7 @@ class FunctionBreakpointsRenderer implements IRenderer<IFunctionBreakpoint, IBas
 		// noop
 	}
 
-	static ID = 'functionbreakpoints';
+	static readonly ID = 'functionbreakpoints';
 
 	get templateId() {
 		return FunctionBreakpointsRenderer.ID;
@@ -429,7 +429,7 @@ class FunctionBreakpointInputRenderer implements IRenderer<IFunctionBreakpoint,
 		// noop
 	}
 
-	static ID = 'functionbreakpointinput';
+	static readonly ID = 'functionbreakpointinput';
 
 	get templateId() {
 		return FunctionBreakpointInputRenderer.ID;

src/vs/workbench/parts/debug/electron-browser/electronDebugActions.ts

@@ -13,7 +13,7 @@ import { IDebugService, IStackFrame } from 'vs/workbench/parts/debug/common/debu
 import { clipboard } from 'electron';
 
 export class CopyValueAction extends Action {
-	static ID = 'workbench.debug.viewlet.action.copyValue';
+	static readonly ID = 'workbench.debug.viewlet.action.copyValue';
 	static LABEL = nls.localize('copyValue', "Copy Value");
 
 	constructor(id: string, label: string, private value: any, @IDebugService private debugService: IDebugService) {
@@ -35,7 +35,7 @@ export class CopyValueAction extends Action {
 }
 
 export class CopyAction extends Action {
-	static ID = 'workbench.debug.action.copy';
+	static readonly ID = 'workbench.debug.action.copy';
 	static LABEL = nls.localize('copy', "Copy");
 
 	public run(): TPromise<any> {
@@ -45,7 +45,7 @@ export class CopyAction extends Action {
 }
 
 export class CopyAllAction extends Action {
-	static ID = 'workbench.debug.action.copyAll';
+	static readonly ID = 'workbench.debug.action.copyAll';
 	static LABEL = nls.localize('copyAll', "Copy All");
 
 	constructor(id: string, label: string, private tree: ITree) {
@@ -69,7 +69,7 @@ export class CopyAllAction extends Action {
 }
 
 export class CopyStackTraceAction extends Action {
-	static ID = 'workbench.action.debug.copyStackTrace';
+	static readonly ID = 'workbench.action.debug.copyStackTrace';
 	static LABEL = nls.localize('copyStackTrace', "Copy Call Stack");
 
 	public run(frame: IStackFrame): TPromise<any> {

src/vs/workbench/parts/extensions/browser/extensionsActions.ts

@@ -350,7 +350,7 @@ export class DropDownMenuActionItem extends ActionItem {
 
 export class ManageExtensionAction extends Action {
 
-	static ID = 'extensions.manage';
+	static readonly ID = 'extensions.manage';
 
 	private static readonly Class = 'extension-action manage';
 	private static readonly HideManageExtensionClass = `${ManageExtensionAction.Class} hide`;
@@ -413,7 +413,7 @@ export class ManageExtensionAction extends Action {
 
 export class EnableForWorkspaceAction extends Action implements IExtensionAction {
 
-	static ID = 'extensions.enableForWorkspace';
+	static readonly ID = 'extensions.enableForWorkspace';
 	static LABEL = localize('enableForWorkspaceAction', "Enable (Workspace)");
 
 	private disposables: IDisposable[] = [];
@@ -453,7 +453,7 @@ export class EnableForWorkspaceAction extends Action implements IExtensionAction
 
 export class EnableGloballyAction extends Action implements IExtensionAction {
 
-	static ID = 'extensions.enableGlobally';
+	static readonly ID = 'extensions.enableGlobally';
 	static LABEL = localize('enableGloballyAction', "Enable");
 
 	private disposables: IDisposable[] = [];
@@ -491,7 +491,7 @@ export class EnableGloballyAction extends Action implements IExtensionAction {
 
 export class EnableAction extends Action {
 
-	static ID = 'extensions.enable';
+	static readonly ID = 'extensions.enable';
 	private static readonly EnabledClass = 'extension-action prominent enable';
 	private static readonly DisabledClass = `${EnableAction.EnabledClass} disabled`;
 
@@ -549,7 +549,7 @@ export class EnableAction extends Action {
 
 export class DisableForWorkspaceAction extends Action implements IExtensionAction {
 
-	static ID = 'extensions.disableForWorkspace';
+	static readonly ID = 'extensions.disableForWorkspace';
 	static LABEL = localize('disableForWorkspaceAction', "Disable (Workspace)");
 
 	private disposables: IDisposable[] = [];
@@ -588,7 +588,7 @@ export class DisableForWorkspaceAction extends Action implements IExtensionActio
 
 export class DisableGloballyAction extends Action implements IExtensionAction {
 
-	static ID = 'extensions.disableGlobally';
+	static readonly ID = 'extensions.disableGlobally';
 	static LABEL = localize('disableGloballyAction', "Disable");
 
 	private disposables: IDisposable[] = [];
@@ -625,7 +625,7 @@ export class DisableGloballyAction extends Action implements IExtensionAction {
 
 export class DisableAction extends Action {
 
-	static ID = 'extensions.disable';
+	static readonly ID = 'extensions.disable';
 
 	private static readonly EnabledClass = 'extension-action disable';
 	private static readonly DisabledClass = `${DisableAction.EnabledClass} disabled`;
@@ -680,7 +680,7 @@ export class DisableAction extends Action {
 
 export class CheckForUpdatesAction extends Action {
 
-	static ID = 'workbench.extensions.action.checkForUpdates';
+	static readonly ID = 'workbench.extensions.action.checkForUpdates';
 	static LABEL = localize('checkForUpdates', "Check for Updates");
 
 	constructor(
@@ -720,7 +720,7 @@ export class ToggleAutoUpdateAction extends Action {
 
 export class EnableAutoUpdateAction extends ToggleAutoUpdateAction {
 
-	static ID = 'workbench.extensions.action.enableAutoUpdate';
+	static readonly ID = 'workbench.extensions.action.enableAutoUpdate';
 	static LABEL = localize('enableAutoUpdate', "Enable Auto Updating Extensions");
 
 	constructor(
@@ -734,7 +734,7 @@ export class EnableAutoUpdateAction extends ToggleAutoUpdateAction {
 
 export class DisableAutoUpdateAction extends ToggleAutoUpdateAction {
 
-	static ID = 'workbench.extensions.action.disableAutoUpdate';
+	static readonly ID = 'workbench.extensions.action.disableAutoUpdate';
 	static LABEL = localize('disableAutoUpdate', "Disable Auto Updating Extensions");
 
 	constructor(
@@ -748,7 +748,7 @@ export class DisableAutoUpdateAction extends ToggleAutoUpdateAction {
 
 export class UpdateAllAction extends Action {
 
-	static ID = 'workbench.extensions.action.updateAllExtensions';
+	static readonly ID = 'workbench.extensions.action.updateAllExtensions';
 	static LABEL = localize('updateAll', "Update All Extensions");
 
 	private disposables: IDisposable[] = [];
@@ -899,7 +899,7 @@ export class InstallExtensionsAction extends OpenExtensionsViewletAction {
 
 export class ShowEnabledExtensionsAction extends Action {
 
-	static ID = 'workbench.extensions.action.showEnabledExtensions';
+	static readonly ID = 'workbench.extensions.action.showEnabledExtensions';
 	static LABEL = localize('showEnabledExtensions', 'Show Enabled Extensions');
 
 	constructor(
@@ -922,7 +922,7 @@ export class ShowEnabledExtensionsAction extends Action {
 
 export class ShowInstalledExtensionsAction extends Action {
 
-	static ID = 'workbench.extensions.action.showInstalledExtensions';
+	static readonly ID = 'workbench.extensions.action.showInstalledExtensions';
 	static LABEL = localize('showInstalledExtensions', "Show Installed Extensions");
 
 	constructor(
@@ -945,7 +945,7 @@ export class ShowInstalledExtensionsAction extends Action {
 
 export class ShowDisabledExtensionsAction extends Action {
 
-	static ID = 'workbench.extensions.action.showDisabledExtensions';
+	static readonly ID = 'workbench.extensions.action.showDisabledExtensions';
 	static LABEL = localize('showDisabledExtensions', "Show Disabled Extensions");
 
 	constructor(
@@ -968,7 +968,7 @@ export class ShowDisabledExtensionsAction extends Action {
 
 export class ClearExtensionsInputAction extends Action {
 
-	static ID = 'workbench.extensions.action.clearExtensionsInput';
+	static readonly ID = 'workbench.extensions.action.clearExtensionsInput';
 	static LABEL = localize('clearExtensionsInput', "Clear Extensions Input");
 
 	private disposables: IDisposable[] = [];
@@ -1004,7 +1004,7 @@ export class ClearExtensionsInputAction extends Action {
 
 export class ShowOutdatedExtensionsAction extends Action {
 
-	static ID = 'workbench.extensions.action.listOutdatedExtensions';
+	static readonly ID = 'workbench.extensions.action.listOutdatedExtensions';
 	static LABEL = localize('showOutdatedExtensions', "Show Outdated Extensions");
 
 	constructor(
@@ -1027,7 +1027,7 @@ export class ShowOutdatedExtensionsAction extends Action {
 
 export class ShowPopularExtensionsAction extends Action {
 
-	static ID = 'workbench.extensions.action.showPopularExtensions';
+	static readonly ID = 'workbench.extensions.action.showPopularExtensions';
 	static LABEL = localize('showPopularExtensions', "Show Popular Extensions");
 
 	constructor(
@@ -1050,7 +1050,7 @@ export class ShowPopularExtensionsAction extends Action {
 
 export class ShowRecommendedExtensionsAction extends Action {
 
-	static ID = 'workbench.extensions.action.showRecommendedExtensions';
+	static readonly ID = 'workbench.extensions.action.showRecommendedExtensions';
 	static LABEL = localize('showRecommendedExtensions', "Show Recommended Extensions");
 
 	constructor(
@@ -1073,7 +1073,7 @@ export class ShowRecommendedExtensionsAction extends Action {
 
 export class InstallWorkspaceRecommendedExtensionsAction extends Action {
 
-	static ID = 'workbench.extensions.action.installWorkspaceRecommendedExtensions';
+	static readonly ID = 'workbench.extensions.action.installWorkspaceRecommendedExtensions';
 	static LABEL = localize('installWorkspaceRecommendedExtensions', "Install All Workspace Recommended Extensions");
 
 	private disposables: IDisposable[] = [];
@@ -1150,7 +1150,7 @@ export class InstallWorkspaceRecommendedExtensionsAction extends Action {
 
 export class InstallRecommendedExtensionAction extends Action {
 
-	static ID = 'workbench.extensions.action.installRecommendedExtension';
+	static readonly ID = 'workbench.extensions.action.installRecommendedExtension';
 	static LABEL = localize('installRecommendedExtension', "Install Recommended Extension");
 
 	private extensionId: string;
@@ -1200,7 +1200,7 @@ export class InstallRecommendedExtensionAction extends Action {
 
 export class ShowRecommendedKeymapExtensionsAction extends Action {
 
-	static ID = 'workbench.extensions.action.showRecommendedKeymapExtensions';
+	static readonly ID = 'workbench.extensions.action.showRecommendedKeymapExtensions';
 	static SHORT_LABEL = localize('showRecommendedKeymapExtensionsShort', "Keymaps");
 
 	constructor(
@@ -1223,7 +1223,7 @@ export class ShowRecommendedKeymapExtensionsAction extends Action {
 
 export class ShowLanguageExtensionsAction extends Action {
 
-	static ID = 'workbench.extensions.action.showLanguageExtensions';
+	static readonly ID = 'workbench.extensions.action.showLanguageExtensions';
 	static SHORT_LABEL = localize('showLanguageExtensionsShort', "Language Extensions");
 
 	constructor(
@@ -1246,7 +1246,7 @@ export class ShowLanguageExtensionsAction extends Action {
 
 export class ShowAzureExtensionsAction extends Action {
 
-	static ID = 'workbench.extensions.action.showAzureExtensions';
+	static readonly ID = 'workbench.extensions.action.showAzureExtensions';
 	static SHORT_LABEL = localize('showAzureExtensionsShort', "Azure Extensions");
 
 	constructor(
@@ -1445,7 +1445,7 @@ export abstract class AbstractConfigureRecommendedExtensionsAction extends Actio
 
 export class ConfigureWorkspaceRecommendedExtensionsAction extends AbstractConfigureRecommendedExtensionsAction {
 
-	static ID = 'workbench.extensions.action.configureWorkspaceRecommendedExtensions';
+	static readonly ID = 'workbench.extensions.action.configureWorkspaceRecommendedExtensions';
 	static LABEL = localize('configureWorkspaceRecommendedExtensions', "Configure Recommended Extensions (Workspace)");
 
 	private disposables: IDisposable[] = [];
@@ -1486,7 +1486,7 @@ export class ConfigureWorkspaceRecommendedExtensionsAction extends AbstractConfi
 
 export class ConfigureWorkspaceFolderRecommendedExtensionsAction extends AbstractConfigureRecommendedExtensionsAction {
 
-	static ID = 'workbench.extensions.action.configureWorkspaceFolderRecommendedExtensions';
+	static readonly ID = 'workbench.extensions.action.configureWorkspaceFolderRecommendedExtensions';
 	static LABEL = localize('configureWorkspaceFolderRecommendedExtensions', "Configure Recommended Extensions (Workspace Folder)");
 
 	private disposables: IDisposable[] = [];
@@ -1555,7 +1555,7 @@ export class BuiltinStatusLabelAction extends Action {
 
 export class DisableAllAction extends Action {
 
-	static ID = 'workbench.extensions.action.disableAll';
+	static readonly ID = 'workbench.extensions.action.disableAll';
 	static LABEL = localize('disableAll', "Disable All Installed Extensions");
 
 	private disposables: IDisposable[] = [];
@@ -1585,7 +1585,7 @@ export class DisableAllAction extends Action {
 
 export class DisableAllWorkpsaceAction extends Action {
 
-	static ID = 'workbench.extensions.action.disableAllWorkspace';
+	static readonly ID = 'workbench.extensions.action.disableAllWorkspace';
 	static LABEL = localize('disableAllWorkspace', "Disable All Installed Extensions for this Workspace");
 
 	private disposables: IDisposable[] = [];
@@ -1617,7 +1617,7 @@ export class DisableAllWorkpsaceAction extends Action {
 
 export class EnableAllAction extends Action {
 
-	static ID = 'workbench.extensions.action.enableAll';
+	static readonly ID = 'workbench.extensions.action.enableAll';
 	static LABEL = localize('enableAll', "Enable All Installed Extensions");
 
 	private disposables: IDisposable[] = [];
@@ -1648,7 +1648,7 @@ export class EnableAllAction extends Action {
 
 export class EnableAllWorkpsaceAction extends Action {
 
-	static ID = 'workbench.extensions.action.enableAllWorkspace';
+	static readonly ID = 'workbench.extensions.action.enableAllWorkspace';
 	static LABEL = localize('enableAllWorkspace', "Enable All Installed Extensions for this Workspace");
 
 	private disposables: IDisposable[] = [];

src/vs/workbench/parts/extensions/common/extensionsInput.ts

@@ -13,7 +13,7 @@ import URI from 'vs/base/common/uri';
 
 export class ExtensionsInput extends EditorInput {
 
-	static ID = 'workbench.extensions.input2';
+	static readonly ID = 'workbench.extensions.input2';
 	get extension(): IExtension { return this._extension; }
 
 	constructor(private _extension: IExtension) {

src/vs/workbench/parts/extensions/electron-browser/extensionsActions.ts

@@ -20,7 +20,7 @@ import { mnemonicButtonLabel } from 'vs/base/common/labels';
 
 export class OpenExtensionsFolderAction extends Action {
 
-	static ID = 'workbench.extensions.action.openExtensionsFolder';
+	static readonly ID = 'workbench.extensions.action.openExtensionsFolder';
 	static LABEL = localize('openExtensionsFolder', "Open Extensions Folder");
 
 	constructor(
@@ -51,7 +51,7 @@ export class OpenExtensionsFolderAction extends Action {
 
 export class InstallVSIXAction extends Action {
 
-	static ID = 'workbench.extensions.action.installVSIX';
+	static readonly ID = 'workbench.extensions.action.installVSIX';
 	static LABEL = localize('installVSIX', "Install from VSIX...");
 
 	constructor(

src/vs/workbench/parts/extensions/electron-browser/runtimeExtensionsEditor.ts

@@ -413,7 +413,7 @@ export class RuntimeExtensionsEditor extends BaseEditor {
 
 export class RuntimeExtensionsInput extends EditorInput {
 
-	static ID = 'workbench.runtimeExtensions.input';
+	static readonly ID = 'workbench.runtimeExtensions.input';
 
 	constructor() {
 		super();
@@ -451,7 +451,7 @@ export class RuntimeExtensionsInput extends EditorInput {
 }
 
 export class ShowRuntimeExtensionsAction extends Action {
-	static ID = 'workbench.action.showRuntimeExtensions';
+	static readonly ID = 'workbench.action.showRuntimeExtensions';
 	static LABEL = nls.localize('showRuntimeExtensions', "Show Running Extensions");
 
 	constructor(
@@ -468,7 +468,7 @@ export class ShowRuntimeExtensionsAction extends Action {
 }
 
 class ReportExtensionIssueAction extends Action {
-	static ID = 'workbench.extensions.action.reportExtensionIssue';
+	static readonly ID = 'workbench.extensions.action.reportExtensionIssue';
 	static LABEL = nls.localize('reportExtensionIssue', "Report Issue");
 
 	constructor(
@@ -506,7 +506,7 @@ class ReportExtensionIssueAction extends Action {
 }
 
 class ExtensionHostProfileAction extends Action {
-	static ID = 'workbench.extensions.action.extensionHostProfile';
+	static readonly ID = 'workbench.extensions.action.extensionHostProfile';
 	static LABEL_START = nls.localize('extensionHostProfileStart', "Start Extension Host Profile");
 	static LABEL_STOP = nls.localize('extensionHostProfileStop', "Stop Extension Host Profile");
 	static STOP_CSS_CLASS = 'extension-host-profile-stop';
@@ -549,7 +549,7 @@ class ExtensionHostProfileAction extends Action {
 class SaveExtensionHostProfileAction extends Action {
 
 	static LABEL = nls.localize('saveExtensionHostProfile', "Save Extension Host Profile");
-	static ID = 'workbench.extensions.action.saveExtensionHostProfile';
+	static readonly ID = 'workbench.extensions.action.saveExtensionHostProfile';
 
 	constructor(
 		id: string = SaveExtensionHostProfileAction.ID, label: string = SaveExtensionHostProfileAction.LABEL,

src/vs/workbench/parts/files/electron-browser/views/openEditorsView.ts

@@ -47,7 +47,7 @@ export class OpenEditorsView extends ViewsViewletPanel {
 
 	private static readonly DEFAULT_VISIBLE_OPEN_EDITORS = 9;
 	private static readonly DEFAULT_DYNAMIC_HEIGHT = true;
-	static ID = 'workbench.explorer.openEditorsView';
+	static readonly ID = 'workbench.explorer.openEditorsView';
 	static NAME = nls.localize({ key: 'openEditors', comment: ['Open is an adjective'] }, "Open Editors");
 
 	private model: IEditorStacksModel;
@@ -413,7 +413,7 @@ class OpenEditorsDelegate implements IDelegate<OpenEditor | IEditorGroup> {
 }
 
 class EditorGroupRenderer implements IRenderer<IEditorGroup, IEditorGroupTemplateData> {
-	static ID = 'editorgroup';
+	static readonly ID = 'editorgroup';
 
 	constructor(
 		private keybindingService: IKeybindingService,
@@ -478,7 +478,7 @@ class EditorGroupRenderer implements IRenderer<IEditorGroup, IEditorGroupTemplat
 }
 
 class OpenEditorRenderer implements IRenderer<OpenEditor, IOpenEditorTemplateData> {
-	static ID = 'openeditor';
+	static readonly ID = 'openeditor';
 	public static DRAGGED_OPEN_EDITOR: OpenEditor;
 
 	constructor(

src/vs/workbench/parts/preferences/browser/keybindingsEditorContribution.ts

@@ -356,7 +356,7 @@ export class KeybindingEditorDecorationsRenderer extends Disposable {
 
 class DefineKeybindingCommand extends EditorCommand {
 
-	static ID = 'editor.action.defineKeybinding';
+	static readonly ID = 'editor.action.defineKeybinding';
 
 	constructor() {
 		super({

src/vs/workbench/parts/scm/electron-browser/scm.contribution.ts

@@ -23,7 +23,7 @@ import { IConfigurationRegistry, Extensions as ConfigurationExtensions } from 'v
 
 class OpenSCMViewletAction extends ToggleViewletAction {
 
-	static ID = VIEWLET_ID;
+	static readonly ID = VIEWLET_ID;
 	static LABEL = localize('toggleGitViewlet', "Show Git");
 
 	constructor(id: string, label: string, @IViewletService viewletService: IViewletService, @IWorkbenchEditorService editorService: IWorkbenchEditorService) {

src/vs/workbench/parts/themes/electron-browser/themes.contribution.ts

@@ -28,7 +28,7 @@ import { LIGHT, DARK, HIGH_CONTRAST } from 'vs/platform/theme/common/themeServic
 
 export class SelectColorThemeAction extends Action {
 
-	static ID = 'workbench.action.selectTheme';
+	static readonly ID = 'workbench.action.selectTheme';
 	static LABEL = localize('selectTheme.label', "Color Theme");
 
 	constructor(
@@ -87,7 +87,7 @@ export class SelectColorThemeAction extends Action {
 
 class SelectIconThemeAction extends Action {
 
-	static ID = 'workbench.action.selectIconTheme';
+	static readonly ID = 'workbench.action.selectIconTheme';
 	static LABEL = localize('selectIconTheme.label', "File Icon Theme");
 
 	constructor(
@@ -171,7 +171,7 @@ function toEntries(themes: (IColorTheme | IFileIconTheme)[], label?: string, bor
 
 class GenerateColorThemeAction extends Action {
 
-	static ID = 'workbench.action.generateColorTheme';
+	static readonly ID = 'workbench.action.generateColorTheme';
 	static LABEL = localize('generateColorTheme.label', "Generate Color Theme From Current Settings");
 
 	constructor(

src/vs/workbench/parts/update/electron-browser/releaseNotesInput.ts

@@ -12,7 +12,7 @@ import URI from 'vs/base/common/uri';
 
 export class ReleaseNotesInput extends EditorInput {
 
-	static ID = 'workbench.releaseNotes.input';
+	static readonly ID = 'workbench.releaseNotes.input';
 
 	get version(): string { return this._version; }
 	get text(): string { return this._text; }

src/vs/workbench/parts/update/electron-browser/update.ts

@@ -165,7 +165,7 @@ export class ShowReleaseNotesAction extends AbstractShowReleaseNotesAction {
 
 export class ShowCurrentReleaseNotesAction extends AbstractShowReleaseNotesAction {
 
-	static ID = 'update.showCurrentReleaseNotes';
+	static readonly ID = 'update.showCurrentReleaseNotes';
 	static LABEL = nls.localize('showReleaseNotes', "Show Release Notes");
 
 	constructor(

src/vs/workbench/parts/welcome/page/electron-browser/welcomePage.ts

@@ -585,7 +585,7 @@ function stripVersion(id: string): string {
 
 export class WelcomeInputFactory implements IEditorInputFactory {
 
-	static ID = welcomeInputTypeId;
+	static readonly ID = welcomeInputTypeId;
 
 	public serialize(editorInput: EditorInput): string {
 		return '{}';

src/vs/workbench/parts/welcome/walkThrough/electron-browser/editor/editorWalkThrough.ts

@@ -47,7 +47,7 @@ export class EditorWalkThroughAction extends Action {
 
 export class EditorWalkThroughInputFactory implements IEditorInputFactory {
 
-	static ID = typeId;
+	static readonly ID = typeId;
 
 	public serialize(editorInput: EditorInput): string {
 		return '{}';