Skip to content

Aud demisto/2026 02 12/gitlab pipeline 5716450 Sk test aud split 3 ok #43066

Open
shmuel44 wants to merge 5 commits intomasterfrom
AUD-demisto/2026-02-12/gitlab-pipeline-5716450
Open

Aud demisto/2026 02 12/gitlab pipeline 5716450 Sk test aud split 3 ok #43066
shmuel44 wants to merge 5 commits intomasterfrom
AUD-demisto/2026-02-12/gitlab-pipeline-5716450

Conversation

@shmuel44
Copy link
Contributor

Contributing to Cortex XSOAR Content

Make sure to register your contribution by filling the contribution registration form

The Pull Request will be reviewed only after the contribution registration form is filled.

Status

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

Related Issues

Description

Must have

  • Tests
  • Documentation

@shmuel44 shmuel44 requested a review from BEAdi February 11, 2026 23:43
@shmuel44 shmuel44 self-assigned this Feb 11, 2026
@shmuel44 shmuel44 added auto-update-docker ready-for-pipeline-running Whether the pr is ready for running the whole pipeline, including testing on SAAS machines ready-for-ai-review The PR is ready for reviewing the PR with the AI Reviewer. aud-split-approved For testing added by Shmuel K labels Feb 11, 2026
@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

@content-bot content-bot removed the ready-for-ai-review The PR is ready for reviewing the PR with the AI Reviewer. label Feb 11, 2026
@content-bot
Copy link
Contributor

🤖 Content AI Reviewer: Analysis started. Please wait for results...

@content-bot
Copy link
Contributor

🤖 Content-bot Review Disclaimer

This review was generated by an AI-powered tool and may contain inaccuracies. Please be advised, and we extend our sincere apologies for any inconvenience this may cause.

Copy link
Contributor

@content-bot content-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the contribution! I noticed a recurring formatting inconsistency across several integration YAML files. Please align the indentation of the dockerimage key to match the sibling keys.

@BEAdi please review and approve the results generated by the AI Reviewer by responding 👍 on this comment.

description: Akamai Generic API Call.
name: akamai-generic-api-call-command
dockerimage: demisto/auth-utils:1.0.0.5476008
dockerimage: demisto/auth-utils:1.0.0.6111183
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The dockerimage key appears to be indented with 2 spaces, while the sibling keys (like script on line 2083) use 4 spaces. Please align the indentation.

- contextPath: Domain.Name
description: Name of the domain returned by the query.
dockerimage: demisto/vendors-sdk:1.0.0.5505539
dockerimage: demisto/vendors-sdk:1.0.0.6383083
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The dockerimage key appears to be indented with 2 spaces, while the sibling keys (like runonce on line 2487) use 4 spaces. Please align the indentation.

description: Gets indicators from the feed.
name: plaintext-get-indicators
dockerimage: demisto/py3-tools:1.0.0.6127057
dockerimage: demisto/py3-tools:1.0.0.6978288
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The dockerimage key appears to be indented with 2 spaces, while the sibling keys (like feed on line 222) use 4 spaces. Please align the indentation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

aud-split-approved For testing added by Shmuel K auto-update-docker ready-for-pipeline-running Whether the pr is ready for running the whole pipeline, including testing on SAAS machines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants