Skip to content

Conversation

marceljk
Copy link
Contributor

@marceljk marceljk commented Sep 2, 2025

Description

relates to STACKITTPR-287 and #907

Checklist

  • Issue was linked above
  • Code format was applied: make fmt
  • Examples were added / adjusted (see examples/ directory)
  • Docs are up-to-date: make generate-docs (will be checked by CI)
  • Unit tests got implemented or updated
  • Acceptance tests got implemented or updated (see e.g. here)
  • Unit tests are passing: make test (will be checked by CI)
  • No linter issues: make lint (will be checked by CI)

@marceljk marceljk requested a review from a team as a code owner September 2, 2025 11:08
@marceljk marceljk force-pushed the fix/STACKITTPR-287-error-deleting-network-area-due-to-attached-project branch from 122fb83 to aab1f67 Compare September 2, 2025 11:09
@marceljk
Copy link
Contributor Author

marceljk commented Sep 2, 2025

is on hold because it depends on stackitcloud/stackit-sdk-go#3522

@marceljk marceljk force-pushed the fix/STACKITTPR-287-error-deleting-network-area-due-to-attached-project branch from aab1f67 to 8f94bbc Compare September 8, 2025 07:55
@marceljk marceljk removed the on-hold label Sep 11, 2025
@marceljk marceljk force-pushed the fix/STACKITTPR-287-error-deleting-network-area-due-to-attached-project branch from de08665 to a52e56e Compare September 11, 2025 11:15
Copy link

Merging this branch will decrease overall coverage

Impacted Packages Coverage Δ 🤖
github.com/stackitcloud/terraform-provider-stackit/stackit/internal/services/iaas/networkarea 34.63% (-0.10%) 👎

Coverage by file

Changed files (no unit tests)

Changed File Coverage Δ Total Covered Missed 🤖
github.com/stackitcloud/terraform-provider-stackit/stackit/internal/services/iaas/networkarea/resource.go 38.80% (-0.13%) 299 (+1) 116 183 (+1) 👎

Please note that the "Total", "Covered", and "Missed" counts above refer to code statements instead of lines of code. The value in brackets refers to the test coverage of that file in the old version of the code.

@marceljk marceljk merged commit 4a7106c into main Sep 11, 2025
5 checks passed
@marceljk marceljk deleted the fix/STACKITTPR-287-error-deleting-network-area-due-to-attached-project branch September 11, 2025 11:26
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