Skip to content

Implement a context content dumping mechanism #3893

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.

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

Merged
merged 7 commits into from
Feb 21, 2024

Conversation

Dakostu
Copy link

@Dakostu Dakostu commented Jan 31, 2024

This change implements a context content dumping mechanism for debugging purposes.

@Dakostu Dakostu added the feature New functionality label Jan 31, 2024
@Dakostu Dakostu requested a review from a team January 31, 2024 13:26
@Dakostu Dakostu force-pushed the topic/dump-context-contents branch 2 times, most recently from 22289b2 to eebdd48 Compare January 31, 2024 13:41
@Dakostu Dakostu force-pushed the topic/dump-context-contents branch 4 times, most recently from b51149c to e0f2fe1 Compare February 19, 2024 15:48
@Dakostu Dakostu marked this pull request as ready for review February 19, 2024 15:52
@Dakostu Dakostu force-pushed the topic/dump-context-contents branch 2 times, most recently from 49d4d46 to d81763c Compare February 20, 2024 10:22
Copy link
Member

@dominiklohmann dominiklohmann left a comment

Choose a reason for hiding this comment

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

Some remaining notes, but approving already.

@Dakostu Dakostu force-pushed the topic/dump-context-contents branch 2 times, most recently from fb8b313 to 1f80dc4 Compare February 20, 2024 13:58
@Dakostu Dakostu enabled auto-merge February 20, 2024 13:59
@Dakostu Dakostu force-pushed the topic/dump-context-contents branch from 1f80dc4 to b8b5cc9 Compare February 20, 2024 15:19
@Dakostu Dakostu merged commit f1f8833 into main Feb 21, 2024
@Dakostu Dakostu deleted the topic/dump-context-contents branch February 21, 2024 07:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants