Fix type errors and enable pre commit #655
benjibcci.yml
on: pull_request
Annotations
7 errors
|
Lint & Type Check
Process completed with exit code 1.
|
|
Lint & Type Check
5 errors
|
|
Lint & Type Check:
eval_protocol/pytest/default_langchain_rollout_processor.py#L94
Argument of type "BaseMessage" cannot be assigned to parameter "msg" of type "BaseMessage" in function "serialize_lc_message_to_ep"
"eval_protocol.pytest.default_langchain_rollout_processor.BaseMessage" is not assignable to "langchain_core.messages.base.BaseMessage" (reportArgumentType)
|
|
Lint & Type Check:
eval_protocol/pytest/default_langchain_rollout_processor.py#L35
Type "type[langchain_core.messages.human.HumanMessage]" is not assignable to declared type "type[eval_protocol.pytest.default_langchain_rollout_processor.LangGraphRolloutProcessor.HumanMessage]"
"langchain_core.messages.human.HumanMessage" is not assignable to "eval_protocol.pytest.default_langchain_rollout_processor.LangGraphRolloutProcessor.HumanMessage"
Type "type[langchain_core.messages.human.HumanMessage]" is not assignable to type "type[eval_protocol.pytest.default_langchain_rollout_processor.LangGraphRolloutProcessor.HumanMessage]" (reportAssignmentType)
|
|
Lint & Type Check:
eval_protocol/pytest/default_langchain_rollout_processor.py#L5
Type "type[langchain_core.messages.base.BaseMessage]" is not assignable to declared type "type[eval_protocol.pytest.default_langchain_rollout_processor.BaseMessage]"
"langchain_core.messages.base.BaseMessage" is not assignable to "eval_protocol.pytest.default_langchain_rollout_processor.BaseMessage"
Type "type[langchain_core.messages.base.BaseMessage]" is not assignable to type "type[eval_protocol.pytest.default_langchain_rollout_processor.BaseMessage]" (reportAssignmentType)
|
|
Lint & Type Check:
eval_protocol/adapters/langfuse.py#L16
"Langfuse" is not exported from module "langfuse"
Import from "langfuse._client.client" instead (reportPrivateImportUsage)
|
|
Lint & Type Check:
eval_protocol/adapters/bigquery.py#L197
Except clause is unreachable because exception is already handled
"Exception" is a subclass of "Exception" (reportUnusedExcept)
|