Skip to content

Revert "[gprpp] Add ValidationErrors::message retrieval"#32767

Merged
ctiller merged 1 commit intomasterfrom
revert-32761-validation-message
Apr 1, 2023
Merged

Revert "[gprpp] Add ValidationErrors::message retrieval"#32767
ctiller merged 1 commit intomasterfrom
revert-32761-validation-message

Conversation

@ctiller
Copy link
Member

@ctiller ctiller commented Apr 1, 2023

Reverts #32761

ctiller added a commit that referenced this pull request Apr 1, 2023
… resolver." (#32768)

Reverts #32632

Needs to be rolled back for #32767 to land
@ctiller ctiller merged commit 4f444aa into master Apr 1, 2023
@ctiller ctiller deleted the revert-32761-validation-message branch April 1, 2023 04:43
@copybara-service copybara-service bot added the imported Specifies if the PR has been imported to the internal repository label Apr 1, 2023
drfloob added a commit to drfloob/grpc that referenced this pull request Apr 1, 2023
drfloob added a commit to drfloob/grpc that referenced this pull request Apr 1, 2023
drfloob added a commit that referenced this pull request Apr 10, 2023
…val" (#32767)". (#32771)

This reverts commit 4f444aa, which
needed a cherrypick.
XuanWang-Amos pushed a commit to XuanWang-Amos/grpc that referenced this pull request May 1, 2023
XuanWang-Amos pushed a commit to XuanWang-Amos/grpc that referenced this pull request May 1, 2023
XuanWang-Amos pushed a commit to XuanWang-Amos/grpc that referenced this pull request May 1, 2023
wanlin31 pushed a commit that referenced this pull request May 18, 2023
… resolver." (#32768)

Reverts #32632

Needs to be rolled back for #32767 to land
wanlin31 pushed a commit that referenced this pull request May 18, 2023
…val" (#32767)". (#32771)

This reverts commit 4f444aa, which
needed a cherrypick.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bloat/none imported Specifies if the PR has been imported to the internal repository lang/c++ lang/core per-call-memory/neutral per-channel-memory/neutral release notes: no Indicates if PR should not be in release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants