Skip to content

add isInternal to searchIndicatorsAgentix and SearchIssuesAgentix #40859

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 17 commits into
base: master
Choose a base branch
from
Draft

add isInternal to searchIndicatorsAgentix and SearchIssuesAgentix #40859

wants to merge 17 commits into from
+8 −1

Conversation

itssapir
Copy link
Contributor

@itssapir itssapir commented Aug 7, 2025

Status

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

Related Issues

related: CRTX-188012

Description

Add isInternal to Agentix-only scripts.

Must have

  • Tests
  • Documentation

@itssapir itssapir added bypass.url Whether to create build bucket, add this label for marketplace.bootstrap.bypass.url ready-for-pipeline-running Whether the pr is ready for running the whole pipeline, including testing on SAAS machines labels Aug 7, 2025
@github-actions GitHub Actions
Copy link

github-actions bot commented Aug 7, 2025 •
edited
Loading

Coverage

Coverage Report
FileStmtsMissCoverMissing
Packs/CommonScripts/Scripts/SearchIndicatorAgentix
   SearchIndicatorAgentix.py34197%28
Packs/CommonScripts/Scripts/SearchIssues
   SearchIssues.py104991%42, 68, 101–102, 139, 157–160
TOTAL1381092% 

Tests Skipped Failures Errors Time
46 0 💤 0 ❌ 0 🔥 5.663s ⏱️

@content-bot
Copy link
Collaborator

Validate summary
The following errors were thrown as a part of this pr: PA114, ST110, RN107.
The following errors cannot be ignored: PA114, ST110, RN107.
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: PA114, RN107.

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
Reviewers
No reviews
Assignees
No one assigned
Labels
bypass.url Whether to create build bucket, add this label for marketplace.bootstrap.bypass.url ready-for-pipeline-running Whether the pr is ready for running the whole pipeline, including testing on SAAS machines
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@itssapir @content-bot