Skip to content

fix: Fix deprovision script when AsUserHeader is empty#672

Open
mwwoda wants to merge 1 commit intomainfrom
sdk-6026-fix-deprovision-script-empty-arg
Open

fix: Fix deprovision script when AsUserHeader is empty#672
mwwoda wants to merge 1 commit intomainfrom
sdk-6026-fix-deprovision-script-empty-arg

Conversation

@mwwoda
Copy link
Copy Markdown
Contributor

@mwwoda mwwoda commented Apr 24, 2026

Omit AsUserHeader in the cli call when empty by using null instead of empty string

@mwwoda mwwoda requested a review from a team April 24, 2026 12:21
@mwwoda mwwoda changed the title fix: Fix deprovision script when AsUserHeader is empty fix: Fix deprovision script when AsUserHeader is empty Apr 24, 2026
@coveralls
Copy link
Copy Markdown

Coverage Report for CI Build 24889199817

Coverage remained the same at 84.999%

Details

  • Coverage remained the same as the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 6025
Covered Lines: 5369
Line Coverage: 89.11%
Relevant Branches: 2141
Covered Branches: 1572
Branch Coverage: 73.42%
Branches in Coverage %: Yes
Coverage Strength: 650.29 hits per line

💛 - Coveralls

Copy link
Copy Markdown
Contributor

@arjankowski arjankowski left a comment

Choose a reason for hiding this comment

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

lgtm

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.

3 participants