Skip to content

Threatconnect bug#43070

Open
merit-maita wants to merge 2 commits intomasterfrom
threatconnect-bug
Open

Threatconnect bug#43070
merit-maita wants to merge 2 commits intomasterfrom
threatconnect-bug

Conversation

@merit-maita
Copy link
Contributor

Status

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

Related Issues

https://jira-dc.paloaltonetworks.com/browse/CIAC-15821

Description

fixed an issue where update indicators was returning an invalid tuple

Must have

  • Tests
  • Documentation

@content-bot
Copy link
Contributor

🤖 AI-Powered Code Review Available

You can leverage AI-powered code review to assist with this PR!

Available Commands:

  • @content-bot start review - Initiate a full AI code review
  • @content-bot re-review - Incremental review for new commits

@github-actions
Copy link

Coverage

Coverage Report
FileStmtsMissCoverMissing
Packs/ThreatConnect/Integrations/ThreatConnectV3
   ThreatConnectV3.py4758781%114, 120, 130, 138, 141–143, 148–149, 155–156, 158, 163, 165–168, 171, 178–181, 197, 201–203, 210–211, 272, 293, 375–381, 455, 460–462, 471–472, 479, 514–515, 709, 775, 780–781, 783–784, 836–837, 851–852, 867, 869, 879, 1917, 2004–2008, 2158–2162, 2273–2277, 2299, 2324–2334
TOTAL4758781% 

Tests Skipped Failures Errors Time
30 0 💤 0 ❌ 0 🔥 2.455s ⏱️

@content-bot
Copy link
Contributor

Validate summary
The following errors were thrown as a part of this pr: DO106.
The following errors cannot be ignored: DO106.
If the AG100 validation in the pre-commit GitHub Action fails, the pull request cannot be force-merged.
The following errors don't run as part of the nightly flow and therefore can be force merged: DO106.

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

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