Skip to content

Add terraform/errors.FrameworkDiagnosticsError - #551

Merged
ulucinar merged 2 commits into
crossplane:mainfrom
ulucinar:diag-err
Nov 5, 2025
Merged

Add terraform/errors.FrameworkDiagnosticsError#551
ulucinar merged 2 commits into
crossplane:mainfrom
ulucinar:diag-err

Conversation

@ulucinar

@ulucinar ulucinar commented Nov 4, 2025

Copy link
Copy Markdown
Collaborator

Description of your changes

Relevant PR: crossplane-contrib/provider-upjet-aws#1900

Adds errors.FrameworkDiagnosticsError to convert from Terraform plugin framework diagnostic error messages to Go errors.

I have:

  • Read and followed Upjet's contribution process.
  • Run make reviewable to ensure this PR is ready for review.
  • Added backport release-x.y labels to auto-backport this PR if necessary.

How has this code been tested

Manually tested against crossplane-contrib/provider-upjet-aws#1900.

Signed-off-by: Alper Rifat Ulucinar <ulucinar@users.noreply.github.com>
…al client

Signed-off-by: Alper Rifat Ulucinar <ulucinar@users.noreply.github.com>

@sergenyalcin sergenyalcin left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks @ulucinar LGTM!

@ulucinar
ulucinar merged commit e6784cf into crossplane:main Nov 5, 2025
6 checks passed
@ulucinar
ulucinar deleted the diag-err branch November 5, 2025 11:41
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