Skip to content

Verify IP Script #40495

New issue

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

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

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Verify IP Script #40495

wants to merge 2 commits into from

Conversation

Ni-Knight
Copy link
Contributor

Status

  • In Progress
  • Ready
  • In Hold - (Reason for hold)

Description

Verifies the input is a valid IP (v6 or v4), also suppurts an array.

Must have

  • Tests
  • Documentation

Copy link

github-actions bot commented Jul 6, 2025

Coverage

Coverage Report
FileStmtsMissCoverMissing
Packs/CommonScripts/Scripts/VerifyValidIP
   VerifyValidIP.py311841%36–38, 40, 55–57, 59, 69–71, 73–77, 79, 89
TOTAL311841% 

Tests Skipped Failures Errors Time
5 0 💤 1 ❌ 0 🔥 2.309s ⏱️

@content-bot
Copy link
Collaborator

Validate summary
The following errors were thrown as a part of this pr: ST110.
The following errors cannot be ignored: ST110.

Verdict: PR can be force merged from validate perspective? ❌

@content-bot
Copy link
Collaborator

This PR is marked as 'Stale' because it has been open for 30 days with no activity, it will be automatically closed in 15 days if no activity will be done. To reset the counter just remove the 'Stale' label or make changes to update this PR. If you wish this PR will never be marked as 'Stale' add the 'Ignore Stale'

@content-bot
Copy link
Collaborator

⚠️ The PR is missing the ready-for-pipeline-running label. Please add the label when the PR is ready in order to proceed.

@content-bot content-bot removed the Stale label Aug 11, 2025
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