Showing 5 of 111 total issues
Avoid deeply nested control flow statements. Open
Open
} else if (splits.length === 2 && splits[0].length > 0 && splits[1].length === 0) {
// intermittent range search, something like `250-` inbetween entering valid states
a = b = Number.parseInt(splits[0]);
} else if (splits.length === 3 && splits[0].length === 0 && splits[2].length === 0) {
// intermittent range search, something like `-250-` inbetween entering valid states
Avoid deeply nested control flow statements. Open
Open
if (a > b) {
const c = b;
b = a;
a = c;
}
Avoid deeply nested control flow statements. Open
Open
if (a <= numberValue && numberValue <= b) {
includeElement = true;
}
Consider simplifying this complex logical expression. Open
Open
if ((isNullOrUndefined(type) && id === "id") ||
(!isNullOrUndefined(type) &&
String(type).toUpperCase() === "NAME-URL" ||
String(type).toUpperCase() === "NAME_URL" ||
String(type).toUpperCase() === "URL")
Consider simplifying this complex logical expression. Open
Open
if ((minDiff >= criteriaValue.minAge || criteriaValue.minAge === undefined || criteriaValue.minAge === -1)
&& (maxDiff < criteriaValue.maxAge || criteriaValue.maxAge === undefined || criteriaValue.maxAge === -1)) {
labels.set(criteriaValue.name, new Label(criteriaValue.name, new Tooltip(criteriaValue.description, ""
+ "- <" + link + ">"
+ "\n- Last repository change: " + date.format("YYYY-MM-DDTHH:mm:ss")