fix: mark instance properties as optional correctly

MarshallOfSound · MarshallOfSound · commit 6571659605a4 · 2019-05-02T14:04:08.000-07:00
diff --git a/src/module-declaration.ts b/src/module-declaration.ts
@@ -272,7 +272,8 @@ export const generateModuleDeclaration = (
     module.instanceProperties
       .sort((a, b) => a.name.localeCompare(b.name))
       .forEach(prop => {
-        moduleAPI.push(`${prop.name}: ${utils.typify(prop)};`);
+        const isOptional = !prop.required ? '?' : '';
+        moduleAPI.push(`${prop.name}${isOptional}: ${utils.typify(prop)};`);
       });
   }
 

Original file line number	Diff line number	Diff line change
`@@ -272,7 +272,8 @@ export const generateModuleDeclaration = (`
`272`	`272`	`module.instanceProperties`
`273`	`273`	`.sort((a, b) => a.name.localeCompare(b.name))`
`274`	`274`	`.forEach(prop => {`
`275`		- moduleAPI.push(`${prop.name}: ${utils.typify(prop)};`);
	`275`	`+ const isOptional = !prop.required ? '?' : '';`
	`276`	+ moduleAPI.push(`${prop.name}${isOptional}: ${utils.typify(prop)};`);
`276`	`277`	`});`
`277`	`278`	`}`
`278`	`279`