Skip to content

Add tests for chat sessions #762

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
dersam merged 1 commit into from
Jan 30, 2026
Merged

Add tests for chat sessions #762

dersam merged 1 commit into from
Jan 30, 2026

Conversation

@dersam
Copy link
Contributor

@dersam dersam commented Jan 30, 2026

Closes #732

This was referenced Jan 30, 2026
Merged
Merged
Merged
@dersam Graphite App
Copy link
Contributor Author

dersam commented Jan 30, 2026 •
edited
Loading

This stack of pull requests is managed by Graphite. Learn more about stacking.

@dersam dersam mentioned this pull request Jan 30, 2026
Merged
@dersam dersam force-pushed the 01-30-add_test_for_roast_cogs_chat_output branch from 9a28e10 to 66cf61a Compare January 30, 2026 18:29
@dersam dersam force-pushed the 01-30-add_tests_for_chat_sessions branch from 89d6783 to 47661b9 Compare January 30, 2026 18:29
@dersam dersam force-pushed the 01-30-add_test_for_roast_cogs_chat_output branch 2 times, most recently from 446c693 to 6376634 Compare January 30, 2026 18:33
@dersam dersam force-pushed the 01-30-add_tests_for_chat_sessions branch from 47661b9 to 2aa9f49 Compare January 30, 2026 18:33
@dersam dersam changed the base branch from 01-30-add_test_for_roast_cogs_chat_output to graphite-base/762 January 30, 2026 18:36
@dersam dersam force-pushed the graphite-base/762 branch from 6376634 to b3c10d0 Compare January 30, 2026 18:37
@dersam dersam force-pushed the 01-30-add_tests_for_chat_sessions branch from 2aa9f49 to 91e2b6c Compare January 30, 2026 18:37
@graphite-app graphite-app bot changed the base branch from graphite-base/762 to main January 30, 2026 18:37
@dersam dersam force-pushed the 01-30-add_tests_for_chat_sessions branch from 91e2b6c to 278d17b Compare January 30, 2026 18:37
@dersam dersam merged commit 88628b4 into main Jan 30, 2026
3 checks passed
@dersam Graphite App
Copy link
Contributor Author

dersam commented Jan 30, 2026

Merge activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mathiusj mathiusj mathiusj approved these changes

@nfgrep nfgrep Awaiting requested review from nfgrep nfgrep is a code owner

@juniper-shopify juniper-shopify Awaiting requested review from juniper-shopify juniper-shopify is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

cogs/chat/session.rb

2 participants

@dersam @mathiusj