Skip to content

Commit 1182638

Browse files
committed
refactor: rename validateSelectors to getValidSelectors
1 parent 8e2de54 commit 1182638

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

src/AzureAppConfigurationImpl.ts

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -465,7 +465,7 @@ export class AzureAppConfigurationImpl implements AzureAppConfiguration {
465465
}
466466
}
467467

468-
function validateSelectors(selectors: SettingSelector[]) {
468+
function getValidSelectors(selectors: SettingSelector[]): SettingSelector[] {
469469
// below code deduplicates selectors by keyFilter and labelFilter, the latter selector wins
470470
const uniqueSelectors: SettingSelector[] = [];
471471
for (const selector of selectors) {
@@ -491,19 +491,19 @@ function validateSelectors(selectors: SettingSelector[]) {
491491
});
492492
}
493493

494-
function getValidKeyValueSelectors(selectors?: SettingSelector[]) {
494+
function getValidKeyValueSelectors(selectors?: SettingSelector[]): SettingSelector[] {
495495
if (!selectors || selectors.length === 0) {
496496
// Default selector: key: *, label: \0
497497
return [{ keyFilter: KeyFilter.Any, labelFilter: LabelFilter.Null }];
498498
}
499-
return validateSelectors(selectors);
499+
return getValidSelectors(selectors);
500500
}
501501

502-
function getValidFeatureFlagSelectors(selectors?: SettingSelector[]) {
502+
function getValidFeatureFlagSelectors(selectors?: SettingSelector[]): SettingSelector[] {
503503
if (!selectors || selectors.length === 0) {
504504
// selectors must be explicitly provided.
505505
throw new Error("Feature flag selectors must be provided.");
506506
} else {
507-
return validateSelectors(selectors);
507+
return getValidSelectors(selectors);
508508
}
509509
}

0 commit comments

Comments
 (0)