Skip to content

Conversation

@JonathanHenson
Copy link
Contributor

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@JonathanHenson JonathanHenson requested a review from a team October 15, 2021 19:47
#include <aws/http/request_response.h>
#include <aws/http/status_code.h>
#include <aws/testing/aws_test_allocators.h>
#include <aws/testing/aws_test_harness.h>
Copy link
Contributor

Choose a reason for hiding this comment

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

you could nuke the whole aws_test_allocators.h file from common

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I did

Copy link
Contributor Author

Choose a reason for hiding this comment

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

that's what broke everything

@JonathanHenson JonathanHenson merged commit 7af5e6f into main Oct 15, 2021
@JonathanHenson JonathanHenson deleted the string_path_fixes branch October 15, 2021 20:44
@JonathanHenson
Copy link
Contributor Author

change is approved and has been verified in the failing codebuild job. However, the github status isn't updating. overriding and merging

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.

4 participants