remove now-redundant check for valid python identifiers in completion…

…s now that the same change was made upstream
DetachHead · Nov 6, 2024 · de1267e · de1267e
1 parent c83ffea
commit de1267e
Showing 1 changed file with 0 additions and 3 deletions.
diff --git a/packages/pyright-internal/src/languageService/completionProvider.ts b/packages/pyright-internal/src/languageService/completionProvider.ts
@@ -130,7 +130,6 @@ import {
 } from './completionProviderUtils';
 import { DocumentSymbolCollector } from './documentSymbolCollector';
 import { getAutoImportText, getDocumentationPartsForTypeAndDecl } from './tooltipUtils';
-import { Tokenizer } from '../parser/tokenizer';
 
 namespace Keywords {
     const base: string[] = [
@@ -2893,8 +2892,6 @@ export class CompletionProvider {
         paramDetails.params.forEach((paramInfo) => {
             if (
                 paramInfo.param.name &&
-                // filter out any dataclass field aliases that aren't valid identifiers
-                Tokenizer.isPythonIdentifier(paramInfo.param.name) &&
                 paramInfo.kind !== ParamKind.Positional &&
                 paramInfo.kind !== ParamKind.ExpandedArgs
             ) {