Code Critics fixes:
Design flaws > Variables not referenced
* Removed 'imageContextHolder' from PharoLauncher
Design flaws > Refers to class name instead of "self class"
* Fixed PharoLauncher class>>settingsOn:
* Fixed PhLDeploymentScript>>addToStartupList
* Fixed PhLDeploymentScript>>removeFromStartupList
MODIFY
src/PharoLauncher-Spec.package/PhLDeploymentScript.class/instance/addToStartupList.st
src/PharoLauncher-Spec.package/PhLDeploymentScript.class/instance/addToStartupList.st
public
addToStartupList
- Smalltalk addToStartUpList: PhLDeploymentScript
\ No newline at end of file
+ Smalltalk addToStartUpList: self class
\ No newline at end of file

MODIFY
src/PharoLauncher-Spec.package/PhLDeploymentScript.class/instance/removeFromStartupList.st
src/PharoLauncher-Spec.package/PhLDeploymentScript.class/instance/removeFromStartupList.st
public
removeFromStartupList
- Smalltalk removeFromStartUpList: PhLDeploymentScript
\ No newline at end of file
+ Smalltalk removeFromStartUpList: self class.
\ No newline at end of file

MODIFY
src/PharoLauncher-Spec.package/PharoLauncher.class/properties.json
src/PharoLauncher-Spec.package/PharoLauncher.class/properties.json
"templatesModel",
"statusbarModel",
"quitOnLaunchCheckBoxModel",
- "imageRepository",
- "imageContextHolder"
+ "imageRepository"
],
"name" : "PharoLauncher",
"type" : "normal"

Code Critics fixes: 

Design flaws > Variables not referenced
* Removed 'imageContextHolder' from PharoLauncher

Design flaws > Refers to class name instead of "self class"
* Fixed PharoLauncher class>>settingsOn:
* Fixed PhLDeploymentScript>>addToStartupList
* Fixed PhLDeploymentScript>>removeFromStartupList