Skip to content

Commit 9f52129

Browse files
chore: fix warnings in helper.mjs
1 parent e55dd09 commit 9f52129

File tree

2 files changed

+16
-12
lines changed

2 files changed

+16
-12
lines changed

utils/convert.mjs

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -517,13 +517,6 @@ function cleanUpClassItems(data) {
517517
const flattenOverloads = funcObj => {
518518
const result = {};
519519

520-
const processOverload = overload => {
521-
if (overload.params) {
522-
return Object.values(overload.params).map(param => processParam(param));
523-
}
524-
return overload;
525-
};
526-
527520
// To simplify `parameterData.json`, instead of having a separate field for
528521
// optional parameters, we'll add a ? to the end of parameter type to
529522
// indicate that it's optional.
@@ -538,6 +531,13 @@ function cleanUpClassItems(data) {
538531
return type;
539532
};
540533

534+
const processOverload = overload => {
535+
if (overload.params) {
536+
return Object.values(overload.params).map(param => processParam(param));
537+
}
538+
return overload;
539+
};
540+
541541
// In some cases, even when the arguments are intended to mean different
542542
// things, their types and order are identical. Since the exact meaning
543543
// of the arguments is less important for parameter validation, we'll

utils/helper.mjs

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -276,7 +276,7 @@ export function organizeData(data) {
276276

277277
allData.forEach(entry => {
278278
const { module, submodule, forEntry } = getModuleInfo(entry);
279-
const cassName = normalizeClassNam(forEntry || entry.memberof || 'p5');
279+
const className = normalizeClassName(forEntry || entry.memberof || 'p5');
280280

281281
switch (entry.kind) {
282282
case 'class':
@@ -293,7 +293,7 @@ export function organizeData(data) {
293293
extends: entry.tags?.find(tag => tag.title === 'extends')?.name || null
294294
}; break;
295295
case 'function':
296-
case 'property':
296+
case 'property': {
297297
const overloads = entry.overloads?.map(overload => ({
298298
params: overload.params,
299299
returns: overload.returns,
@@ -315,7 +315,9 @@ export function organizeData(data) {
315315
class: className,
316316
isStatic: entry.path?.[0]?.scope === 'static',
317317
overloads
318-
}); break;
318+
});
319+
break;
320+
}
319321
case 'constant':
320322
case 'typedef':
321323
organized.consts[entry.name] = {
@@ -326,7 +328,8 @@ export function organizeData(data) {
326328
module,
327329
submodule,
328330
class: forEntry || 'p5'
329-
}; break;
331+
};
332+
break;
330333
}
331334
});
332335
return organized;
@@ -373,11 +376,12 @@ export function generateTypeFromTag(param) {
373376
.join(', ');
374377
return `${baseType}<${typeParams}>`;
375378
}
376-
case 'UnionType':
379+
case 'UnionType': {
377380
const unionTypes = param.type.elements
378381
.map(el => generateTypeFromTag({ type: el }))
379382
.join(' | ');
380383
return unionTypes;
384+
}
381385
case 'OptionalType':
382386
return generateTypeFromTag({ type: param.type.expression });
383387
case 'AllLiteral':

0 commit comments

Comments
 (0)