Skip to content

Auto RN: MW/Rename_Scripts#44289

Merged
dantavori merged 5 commits into
masterfrom
MW/Rename_Scripts
May 14, 2026
Merged

Auto RN: MW/Rename_Scripts#44289
dantavori merged 5 commits into
masterfrom
MW/Rename_Scripts

Conversation

@content-bot
Copy link
Copy Markdown
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

@content-bot content-bot added 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. labels May 12, 2026
@content-bot content-bot requested review from mmhw and omerKarkKatz May 12, 2026 18:57
@content-bot
Copy link
Copy Markdown
Contributor Author

🤖 AI-Powered Code Review Available

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

Available Commands:

  • @marketplace-ai-reviewer start review - Initiate a full AI code review
  • @marketplace-ai-reviewer re-review - Incremental review for new commits

@marketplace-ai-reviewer marketplace-ai-reviewer removed the ready-for-ai-review The PR is ready for reviewing the PR with the AI Reviewer. label May 12, 2026
@marketplace-ai-reviewer
Copy link
Copy Markdown
Contributor

🤖 Analysis started. Please wait for results...

marketplace-ai-reviewer

This comment was marked as outdated.

@content-bot

This comment has been minimized.

@mmhw
Copy link
Copy Markdown
Contributor

mmhw commented May 13, 2026

@marketplace-ai-reviewer start review

@mmhw mmhw added docs-approved ForceMerge Forcing the merge of the PR despite the build status labels May 13, 2026
@mmhw
Copy link
Copy Markdown
Contributor

mmhw commented May 13, 2026

Reason for force merge:
BC105
Rename scripts ids.

@marketplace-ai-reviewer
Copy link
Copy Markdown
Contributor

🤖 Analysis started. Please wait for results...

@marketplace-ai-reviewer
Copy link
Copy Markdown
Contributor

🤖 AI 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
Copy Markdown
Contributor

@marketplace-ai-reviewer marketplace-ai-reviewer left a comment

Choose a reason for hiding this comment

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

Hi there! Thanks for your contribution. I've reviewed your PR and noticed a couple of quick updates needed for the pack metadata. Specifically, please ensure the mandatory created field is included and add the vendor name to the keywords list. Thanks again for your work on this!

Additionally, please address the following file-level notes:

  • Packs/Core/pack_metadata.json: - Missing mandatory field: created.
  • Vendor name is missing from keywords.

@JudahSchwartz, @omerKarkKatz, @marketplace-ai-reviewer, @mmhw, @content-bot please review and approve the results generated by the AI Reviewer by responding 👍 on this comment.

@content-bot
Copy link
Copy Markdown
Contributor Author

This PR was automatically updated by a GitHub Action

  • Core pack version was bumped to 3.5.47.

To stop automatic version bumps, add the ignore-auto-bump-version label to the github PR.

@content-bot
Copy link
Copy Markdown
Contributor Author

🔍 AI Triage Report Available

An automated triage report has been generated for this pipeline.

Status: failed
Report ID: 263e3ee0580844a1

📋 Triage Report
💡 Resolutions are available in the full report.

⚠️ AI-generated triage. Validate before acting.

@content-bot
Copy link
Copy Markdown
Contributor Author

Validate summary
The following errors were reported as warnings: GR101, MC101.
The following errors were thrown as a part of this pr: RN107, RN114, BC105.
The following errors can be ignored: RN107, RN114.
The following errors cannot be ignored: BC105.
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: RN107, RN114, BC105.

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

@dantavori dantavori merged commit 4eca893 into master May 14, 2026
20 of 21 checks passed
@dantavori dantavori deleted the MW/Rename_Scripts branch May 14, 2026 12:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs-approved ForceMerge Forcing the merge of the PR despite the build status 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.

5 participants