Skip to content

Commit 2a86372

Browse files
authored
fix(infra,core): nits (#34079)
* Add missing `nits` to allowed PR linting scopes * Ensure `MAJOR.MINOR.PATCH` consistency in admonitions * Ensure valid spacing in admonitions
1 parent 30e2260 commit 2a86372

File tree

3 files changed

+5
-2
lines changed

3 files changed

+5
-2
lines changed

.github/workflows/pr_lint.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -102,6 +102,7 @@ jobs:
102102
qdrant
103103
xai
104104
infra
105+
deps
105106
requireScope: false
106107
disallowScopes: |
107108
release

libs/core/langchain_core/language_models/chat_models.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -333,7 +333,7 @@ class BaseChatModel(BaseLanguageModel[AIMessage], ABC):
333333
[`langchain-openai`](https://pypi.org/project/langchain-openai)) can also use this
334334
field to roll out new content formats in a backward-compatible way.
335335
336-
!!! version-added "Added in `langchain-core` 1.0"
336+
!!! version-added "Added in `langchain-core` 1.0.0"
337337
338338
"""
339339

@@ -349,7 +349,7 @@ class BaseChatModel(BaseLanguageModel[AIMessage], ABC):
349349
Example profile data includes context window sizes, supported modalities, or support
350350
for tool calling, structured output, and other features.
351351
352-
!!! version-added "Added in `langchain-core` 1.1"
352+
!!! version-added "Added in `langchain-core` 1.1.0"
353353
"""
354354

355355
model_config = ConfigDict(

libs/core/langchain_core/messages/utils.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -738,8 +738,10 @@ def trim_messages(
738738
Set to `len` to count the number of **messages** in the chat history.
739739
740740
!!! note
741+
741742
Use `count_tokens_approximately` to get fast, approximate token
742743
counts.
744+
743745
This is recommended for using `trim_messages` on the hot path, where
744746
exact token counting is not necessary.
745747

0 commit comments

Comments
 (0)