Skip to content

fix(empty messages): don't embed in chat history#551

Merged
douglas-reid merged 2 commits intomainfrom
doug/explore-whitespace-filtering
Sep 27, 2023
Merged

fix(empty messages): don't embed in chat history#551
douglas-reid merged 2 commits intomainfrom
doug/explore-whitespace-filtering

Conversation

@douglas-reid
Copy link
Copy Markdown
Contributor

No description provided.

@douglas-reid douglas-reid requested a review from dkolas September 15, 2023 22:38
Copy link
Copy Markdown
Contributor

@dkolas dkolas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This works for me... we may want to throw an error at a lower level in the stack

@douglas-reid douglas-reid marked this pull request as ready for review September 26, 2023 22:35
@douglas-reid douglas-reid changed the title wip(empty messages): don't embed in chat history fix(empty messages): don't embed in chat history Sep 26, 2023
@douglas-reid douglas-reid added this pull request to the merge queue Sep 27, 2023
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Sep 27, 2023
@douglas-reid douglas-reid added this pull request to the merge queue Sep 27, 2023
Merged via the queue into main with commit 844532d Sep 27, 2023
@douglas-reid douglas-reid deleted the doug/explore-whitespace-filtering branch September 28, 2023 02:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants