Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

SAST-Create-report: bug fix: provide default for report_teams param #66

Merged
merged 1 commit into from
Dec 1, 2023
Merged

SAST-Create-report: bug fix: provide default for report_teams param #66

merged 1 commit into from
Dec 1, 2023

Conversation

james-bostock-cx
Copy link
Contributor

We set the default value of the report_teams parameter to an empty array so that when we pipe it to the ForEach-Object we don't get:

+     if ( ! $_.StartsWith("/") ) {
+          ~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (:) [], RuntimeException
    + FullyQualifiedErrorId : InvokeMethodOnNull

@james-bostock-cx
SAST-Create-report: bug fix: provide default for report_teams param
641c6e2 
We set the default value of the report_teams parameter to an empty
array so that when we pipe it to the ForEach-Object we don't get:

+     if ( ! $_.StartsWith("/") ) {
+          ~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (:) [], RuntimeException
    + FullyQualifiedErrorId : InvokeMethodOnNull
@nleach999 nleach999 mentioned this pull request Nov 10, 2023
Open
@nleach999 nleach999 merged commit ca6a2c9 into master Dec 1, 2023
@nleach999 nleach999 deleted the pr-create-sast-report-report-teams-fix branch December 1, 2023 14:05
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@nleach999 nleach999 nleach999 approved these changes

@benjaminstokes benjaminstokes Awaiting requested review from benjaminstokes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@james-bostock-cx @nleach999