Skip to content

cpp-v3 test server#55

Merged
ajewellamz merged 13 commits into
fireegg-test-serversfrom
ajewell/v3
Oct 30, 2025
Merged

cpp-v3 test server#55
ajewellamz merged 13 commits into
fireegg-test-serversfrom
ajewell/v3

Conversation

@ajewellamz

Copy link
Copy Markdown

Issue #, if available:

Description of changes:

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@ajewellamz ajewellamz changed the base branch from main to fireegg-test-servers October 29, 2025 15:36
seebees
seebees previously approved these changes Oct 30, 2025

@seebees seebees left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM


[[source]]
pattern = "aws-sdk-cpp/src/aws-cpp-sdk-s3-encryption/**/*.h"
pattern = "aws-sdk-cpp/src/aws-cpp-sdk-s3-encryption/**/*.cpp aws-sdk-cpp/src/aws-cpp-sdk-s3-encryption/**/*.h"

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I think you want this as 2 source blocks

Comment thread test-server/cpp-v2-transition-server/main.cpp

@seebees seebees left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@ajewellamz ajewellamz merged commit b131869 into fireegg-test-servers Oct 30, 2025
3 checks passed
@ajewellamz ajewellamz deleted the ajewell/v3 branch October 30, 2025 18:25
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