Skip to content

Conversation

@alexott
Copy link
Contributor

@alexott alexott commented Nov 13, 2025

Changes

Also adding more generic functions, i.e. handle of effective fields, etc.

Tests

  • make test run locally
  • relevant change in docs/ folder
  • covered with integration tests in internal/acceptance
  • using Go SDK
  • using TF Plugin Framework
  • has entry in NEXT_CHANGELOG.md file

Also adding more generic functions, i.e. handle of effective fields, etc.
@alexott alexott requested review from a team as code owners November 13, 2025 08:07
@alexott alexott requested review from chrisst and removed request for a team November 13, 2025 08:07
@alexott alexott requested a review from rauchy November 13, 2025 08:07
@github-actions
Copy link

If integration tests don't run automatically, an authorized user can run them manually by following the instructions below:

Trigger:
go/deco-tests-run/terraform

Inputs:

  • PR number: 5212
  • Commit SHA: d437e71a1dbc6aa599a4b92d434b45d1ee68dc29

Checks will be approved automatically on success.

@alexott alexott added exporter TF configuration generator ai-assisted labels Nov 13, 2025
@@ -0,0 +1 @@
package exporter
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this placeholder file intentional? If there are no tests yet for abstractions.go, might be worth either adding tests for copyEffectiveFieldsToInputFieldsWithConverters or removing the empty file.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ok, I'll check with the next PRs - I think that it's used there. If not, will remove it

@alexott alexott added this pull request to the merge queue Nov 26, 2025
Merged via the queue into main with commit aacbf20 Nov 26, 2025
12 checks passed
@alexott alexott deleted the exporter/lakebase branch November 26, 2025 09:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ai-assisted exporter TF configuration generator

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants