Skip to content

Conversation

fapo85
Copy link

@fapo85 fapo85 commented Sep 15, 2025

Description

Add STACKIT Cloud Foundry resources to the terraform provier

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)

Co-authored-by: @akop

@fapo85 fapo85 requested a review from a team as a code owner September 15, 2025 08:38
@fapo85 fapo85 changed the title Feature/scf feat(scf): Add STACKIT Cloud Foundry Sep 15, 2025
@rubenhoenle
Copy link
Member

@fapo85 Please consider squashing these 55 commits into a meaningful commits. If that's not possible it's also fine for me to squash them into a single commit. But I don't see any benefit in having 55 commits here 😄

@fapo85 fapo85 force-pushed the feature/scf branch 2 times, most recently from 2f5df45 to 3625c02 Compare September 15, 2025 13:10
@fapo85 fapo85 closed this Sep 15, 2025
@fapo85 fapo85 reopened this Sep 15, 2025
@fapo85
Copy link
Author

fapo85 commented Sep 15, 2025

@fapo85 Please consider squashing these 55 commits into a meaningful commits. If that's not possible it's also fine for me to squash them into a single commit. But I don't see any benefit in having 55 commits here 😄

done, thanks for the hint

Copy link

github-actions bot commented Oct 2, 2025

This PR was marked as stale after 7 days of inactivity and will be closed after another 7 days of further inactivity. If this PR should be kept open, just add a comment, remove the stale label or push new commits to it.

@github-actions github-actions bot added the Stale PR is marked as stale due to inactivity. label Oct 2, 2025
@marceljk marceljk removed the Stale PR is marked as stale due to inactivity. label Oct 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants