Skip to content

fix(system-messages): merge messages when provider is vllm or when enabled by config#3255

Merged
amitksingh1490 merged 6 commits intomainfrom
fix/merge-messages-when-provider-is-vllm
May 11, 2026
Merged

fix(system-messages): merge messages when provider is vllm or when enabled by config#3255
amitksingh1490 merged 6 commits intomainfrom
fix/merge-messages-when-provider-is-vllm

Conversation

@laststylebender14
Copy link
Copy Markdown
Contributor

@laststylebender14 laststylebender14 commented May 4, 2026

fixes: #3128

@github-actions github-actions Bot added the type: fix Iterations on existing features or infrastructure. label May 4, 2026
@laststylebender14 laststylebender14 marked this pull request as ready for review May 4, 2026 09:08
@amitksingh1490 amitksingh1490 changed the title fix(system-messagees): merge messages when provider is vllm or when enabled by config fix(system-messages): merge messages when provider is vllm or when enabled by config May 11, 2026
@amitksingh1490 amitksingh1490 enabled auto-merge (squash) May 11, 2026 04:12
@amitksingh1490 amitksingh1490 merged commit 5d6be4e into main May 11, 2026
10 checks passed
@amitksingh1490 amitksingh1490 deleted the fix/merge-messages-when-provider-is-vllm branch May 11, 2026 04:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: fix Iterations on existing features or infrastructure.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: System message must be at the beginning. when using vllm

2 participants