Skip to content

Use empty array for findings where no ips were identified#1748

Merged
J12934 merged 1 commit intomainfrom
bugfix/ip-addresses-use-null-over-undefined
Jun 5, 2023
Merged

Use empty array for findings where no ips were identified#1748
J12934 merged 1 commit intomainfrom
bugfix/ip-addresses-use-null-over-undefined

Conversation

@J12934
Copy link
Copy Markdown
Member

@J12934 J12934 commented Jun 5, 2023

Followup for #1701

Checklist

  • Test your changes as thoroughly as possible before you commit them. Preferably, automate your test by unit/integration tests.
  • Make sure that all your commits are signed-off and that you are added to the Contributors file.
  • [] Make sure that all CI finish successfully.
  • Optional (but appreciated): Make sure that all commits are Verified.

Followup for #1701

Signed-off-by: Jannik Hollenbach <jannik.hollenbach@iteratec.com>
@J12934 J12934 added this to the v4.0.0 milestone Jun 5, 2023
@J12934 J12934 requested a review from Ilyesbdlala June 5, 2023 11:21
@J12934 J12934 self-assigned this Jun 5, 2023
@J12934 J12934 added the skip-changelog Skipped from changelog generation label Jun 5, 2023
@github-actions
Copy link
Copy Markdown

github-actions bot commented Jun 5, 2023

MegaLinter status: ⚠️ WARNING

Descriptor Linter Files Fixed Errors Elapsed time
⚠️ JAVASCRIPT eslint 3 1 0.66s
✅ SPELL misspell 3 0 0.05s

See errors details in artifact MegaLinter reports on CI Job page
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff

@J12934 J12934 merged commit 9d2f6e5 into main Jun 5, 2023
@J12934 J12934 deleted the bugfix/ip-addresses-use-null-over-undefined branch June 5, 2023 18:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Bugs findings skip-changelog Skipped from changelog generation

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants