Skip to content

Conversation

rubenhoenle
Copy link
Member

Description

relates to #1234

Checklist

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

@rubenhoenle rubenhoenle self-assigned this Sep 10, 2025
Copy link

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 label Sep 18, 2025
@rubenhoenle rubenhoenle force-pushed the chore/add-string-to-string-flag-parsing-unit-test branch from a791139 to 4e820dc Compare September 18, 2025 16:42
@rubenhoenle rubenhoenle marked this pull request as ready for review September 18, 2025 16:42
@rubenhoenle rubenhoenle requested a review from a team as a code owner September 18, 2025 16:42
@rubenhoenle rubenhoenle removed the Stale label Sep 18, 2025
@rubenhoenle rubenhoenle enabled auto-merge (squash) September 18, 2025 16:47
@marceljk marceljk force-pushed the chore/add-string-to-string-flag-parsing-unit-test branch from 4e820dc to c6bfb0a Compare September 19, 2025 15:10
@rubenhoenle rubenhoenle merged commit 9896dba into main Sep 19, 2025
5 checks passed
@rubenhoenle rubenhoenle deleted the chore/add-string-to-string-flag-parsing-unit-test branch September 19, 2025 15:15
Copy link

Merging this branch will increase overall coverage

Impacted Packages Coverage Δ 🤖
github.com/stackitcloud/stackit-cli/internal/pkg/flags 58.29% (+2.51%) 👍

Coverage by file

Changed unit test files

  • github.com/stackitcloud/stackit-cli/internal/pkg/flags/flag_to_value_test.go

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