Remove PyCharm specific comments

These were not needed any more and partially outdated anyway
after switching to VS Code and relying on ruff and mypy.

Author: Cito

src/graphql/execution/collect_fields.py

@@ -319,7 +319,6 @@ def does_fragment_condition_match(
     if conditional_type is type_:
         return True
     if is_abstract_type(conditional_type):
-        # noinspection PyTypeChecker
         return schema.is_sub_type(conditional_type, type_)
     return False
 

src/graphql/execution/execute.py

@@ -119,7 +119,6 @@
 try:  # pragma: no cover
     anext  # noqa: B018  # pyright: ignore
 except NameError:  # pragma: no cover (Python < 3.10)
-    # noinspection PyShadowingBuiltins
     async def anext(iterator: AsyncIterator) -> Any:
         """Return the next item from an async iterator."""
         return await iterator.__anext__()
@@ -445,7 +444,6 @@ def execute_root_grouped_field_set(
         defer_map: RefMap[DeferUsage, DeferredFragmentRecord] | None,
     ) -> AwaitableOrValue[GraphQLWrappedResult[dict[str, Any]]]:
         """Execute the root grouped field set."""
-        # noinspection PyTypeChecker
         return (
             self.execute_fields_serially
             if operation == OperationType.MUTATION
@@ -506,7 +504,6 @@ async def set_result() -> GraphQLWrappedResult[dict[str, Any]]:
             graphql_wrapped_result.add_increments(resolved.increments)
             return graphql_wrapped_result
 
-        # noinspection PyTypeChecker
         return async_reduce(
             reducer, grouped_field_set.items(), GraphQLWrappedResult({})
         )
@@ -647,7 +644,6 @@ def execute_field(
                 defer_map,
             )
             if self.is_awaitable(completed):
-                # noinspection PyShadowingNames
                 async def await_completed() -> Any:
                     try:
                         return await completed
@@ -1168,7 +1164,6 @@ def complete_iterable_value(
         if not awaitable_indices:
             return graphql_wrapped_result
 
-        # noinspection PyShadowingNames
         async def get_completed_results() -> GraphQLWrappedResult[list[Any]]:
             if len(awaitable_indices) == 1:
                 # If there is only one index, avoid the overhead of parallelization.
@@ -1216,7 +1211,6 @@ def complete_list_item_value(
             )
 
             if is_awaitable(completed_item):
-                # noinspection PyShadowingNames
                 async def await_completed() -> Any:
                     try:
                         resolved = await completed_item  # type: ignore
@@ -1421,7 +1415,6 @@ def ensure_valid_runtime_type(
             )
             raise GraphQLError(msg, to_nodes(field_group))
 
-        # noinspection PyTypeChecker
         return runtime_type
 
     def complete_object_value(
@@ -2674,7 +2667,6 @@ def execute_subscription(
 
         result = resolve_fn(context.root_value, info, **args)
         if context.is_awaitable(result):
-            # noinspection PyShadowingNames
             async def await_result() -> AsyncIterable[Any]:
                 try:
                     return assert_event_stream(await result)

src/graphql/language/ast.py

@@ -404,7 +404,6 @@ def __copy__(self) -> Node:
 
     def __deepcopy__(self, memo: dict) -> Node:
         """Create a deep copy of the node"""
-        # noinspection PyArgumentList
         return self.__class__(
             **{key: deepcopy(getattr(self, key), memo) for key in self.keys}
         )
@@ -422,7 +421,6 @@ def __init_subclass__(cls) -> None:
         cls.kind = camel_to_snake(name)
         keys: list[str] = []
         for base in cls.__bases__:
-            # noinspection PyUnresolvedReferences
             keys.extend(base.keys)  # type: ignore
         keys.extend(cls.__slots__)
         cls.keys = tuple(keys)

src/graphql/language/parser.py

@@ -582,7 +582,6 @@ def parse_list(self, is_const: bool) -> ListValueNode:
         """ListValue[Const]"""
         start = self._lexer.token
         item = partial(self.parse_value_literal, is_const)
-        # noinspection PyTypeChecker
         return ListValueNode(
             values=self.any(TokenKind.BRACKET_L, item, TokenKind.BRACKET_R),
             loc=self.loc(start),

src/graphql/pyutils/suggestion_list.py

@@ -24,7 +24,6 @@ def suggestion_list(input_: str, options: Collection[str]) -> list[str]:
         if distance is not None:
             options_by_distance[option] = distance
 
-    # noinspection PyShadowingNames
     return sorted(
         options_by_distance,
         key=lambda option: (

src/graphql/type/definition.py

@@ -432,7 +432,6 @@ def parse_literal(
 
     def to_kwargs(self) -> GraphQLScalarTypeKwargs:
         """Get corresponding arguments."""
-        # noinspection PyArgumentList
         return GraphQLScalarTypeKwargs(
             super().to_kwargs(),  # type: ignore
             serialize=None
@@ -771,7 +770,6 @@ def __init__(
 
     def to_kwargs(self) -> GraphQLObjectTypeKwargs:
         """Get corresponding arguments."""
-        # noinspection PyArgumentList
         return GraphQLObjectTypeKwargs(
             super().to_kwargs(),  # type: ignore
             fields=self.fields.copy(),
@@ -876,7 +874,6 @@ def __init__(
 
     def to_kwargs(self) -> GraphQLInterfaceTypeKwargs:
         """Get corresponding arguments."""
-        # noinspection PyArgumentList
         return GraphQLInterfaceTypeKwargs(
             super().to_kwargs(),  # type: ignore
             fields=self.fields.copy(),
@@ -981,7 +978,6 @@ def __init__(
 
     def to_kwargs(self) -> GraphQLUnionTypeKwargs:
         """Get corresponding arguments."""
-        # noinspection PyArgumentList
         return GraphQLUnionTypeKwargs(
             super().to_kwargs(),  # type: ignore
             types=self.types,
@@ -1112,7 +1108,6 @@ def __init__(
 
     def to_kwargs(self) -> GraphQLEnumTypeKwargs:
         """Get corresponding arguments."""
-        # noinspection PyArgumentList
         return GraphQLEnumTypeKwargs(
             super().to_kwargs(),  # type: ignore
             values=self.values.copy(),
@@ -1341,7 +1336,6 @@ def out_type(value: dict[str, Any]) -> Any:
 
     def to_kwargs(self) -> GraphQLInputObjectTypeKwargs:
         """Get corresponding arguments."""
-        # noinspection PyArgumentList
         return GraphQLInputObjectTypeKwargs(
             super().to_kwargs(),  # type: ignore
             fields=self.fields.copy(),

src/graphql/type/introspection.py

@@ -154,7 +154,6 @@ def is_repeatable(directive, _info):
     def locations(directive, _info):
         return directive.locations
 
-    # noinspection PyPep8Naming
     @staticmethod
     def args(directive, _info, includeDeprecated=False):
         items = directive.args.items()
@@ -343,7 +342,6 @@ def description(type_, _info):
     def specified_by_url(type_, _info):
         return getattr(type_, "specified_by_url", None)
 
-    # noinspection PyPep8Naming
     @staticmethod
     def fields(type_, _info, includeDeprecated=False):
         if not (is_object_type(type_) or is_interface_type(type_)):
@@ -369,7 +367,6 @@ def possible_types(type_, info):
             info.schema.get_possible_types(type_) if is_abstract_type(type_) else None
         )
 
-    # noinspection PyPep8Naming
     @staticmethod
     def enum_values(type_, _info, includeDeprecated=False):
         if not is_enum_type(type_):
@@ -381,7 +378,6 @@ def enum_values(type_, _info, includeDeprecated=False):
             else [item for item in items if item[1].deprecation_reason is None]
         )
 
-    # noinspection PyPep8Naming
     @staticmethod
     def input_fields(type_, _info, includeDeprecated=False):
         if not is_input_object_type(type_):
@@ -450,7 +446,6 @@ def name(item, _info):
     def description(item, _info):
         return item[1].description
 
-    # noinspection PyPep8Naming
     @staticmethod
     def args(item, _info, includeDeprecated=False):
         items = item[1].args.items()

src/graphql/type/validate.py

@@ -57,7 +57,6 @@ def validate_schema(schema: GraphQLSchema) -> list[GraphQLError]:
     assert_schema(schema)
 
     # If this Schema has already been validated, return the previous results.
-    # noinspection PyProtectedMember
     errors = schema._validation_errors  # noqa: SLF001
     if errors is None:
         # Validate the schema, producing a list of errors.

src/graphql/utilities/extend_schema.py

@@ -279,7 +279,6 @@ def extend_schema_args(
             assume_valid=assume_valid,
         )
 
-    # noinspection PyTypeChecker,PyUnresolvedReferences
     def replace_type(self, type_: GraphQLType) -> GraphQLType:
         """Replace a GraphQL type."""
         if is_list_type(type_):
@@ -295,7 +294,6 @@ def replace_named_type(self, type_: GraphQLNamedType) -> GraphQLNamedType:
         # validation with validate_schema() will produce more actionable results.
         return self.type_map[type_.name]
 
-    # noinspection PyShadowingNames
     def replace_directive(self, directive: GraphQLDirective) -> GraphQLDirective:
         """Replace a GraphQL directive."""
         if is_specified_directive(directive):
@@ -351,7 +349,6 @@ def extend_input_object_type_fields(
             **self.build_input_field_map(extensions),
         }
 
-    # noinspection PyShadowingNames
     def extend_input_object_type(
         self,
         type_: GraphQLInputObjectType,
@@ -421,7 +418,6 @@ def extend_object_type_fields(
             **self.build_field_map(extensions),
         }
 
-    # noinspection PyShadowingNames
     def extend_object_type(self, type_: GraphQLObjectType) -> GraphQLObjectType:
         """Extend a GraphQL object type."""
         kwargs = type_.to_kwargs()
@@ -459,7 +455,6 @@ def extend_interface_type_fields(
             **self.build_field_map(extensions),
         }
 
-    # noinspection PyShadowingNames
     def extend_interface_type(
         self, type_: GraphQLInterfaceType
     ) -> GraphQLInterfaceType:
@@ -500,7 +495,6 @@ def extend_union_type(self, type_: GraphQLUnionType) -> GraphQLUnionType:
             ),
         )
 
-    # noinspection PyShadowingNames
     def extend_field(self, field: GraphQLField) -> GraphQLField:
         """Extend a GraphQL field."""
         return GraphQLField(
@@ -520,7 +514,6 @@ def extend_arg(self, arg: GraphQLArgument) -> GraphQLArgument:
             )
         )
 
-    # noinspection PyShadowingNames
     def get_operation_types(
         self, nodes: Collection[SchemaDefinitionNode | SchemaExtensionNode]
     ) -> dict[OperationType, GraphQLNamedType]:
@@ -534,7 +527,6 @@ def get_operation_types(
             for operation_type in node.operation_types or []
         }
 
-    # noinspection PyShadowingNames
     def get_named_type(self, node: NamedTypeNode) -> GraphQLNamedType:
         """Get name GraphQL type for a given named type node."""
         name = node.name.value

src/graphql/utilities/type_comparators.py

@@ -25,12 +25,10 @@ def is_equal_type(type_a: GraphQLType, type_b: GraphQLType) -> bool:
 
     # If either type is non-null, the other must also be non-null.
     if is_non_null_type(type_a) and is_non_null_type(type_b):
-        # noinspection PyUnresolvedReferences
         return is_equal_type(type_a.of_type, type_b.of_type)
 
     # If either type is a list, the other must also be a list.
     if is_list_type(type_a) and is_list_type(type_b):
-        # noinspection PyUnresolvedReferences
         return is_equal_type(type_a.of_type, type_b.of_type)
 
     # Otherwise the types are not equal.