Skip to content

Add "platform" to hidden fields. #40195

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 8 commits into
base: master
Choose a base branch
from

Conversation

jlevypaloalto
Copy link
Contributor

Status

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

Related Issues

fixes: link to the issue

Description

A few sentences describing the overall goals of the pull request's commits.

Must have

  • Tests
  • Documentation

Copy link

github-actions bot commented Jun 5, 2025

Your contributed mnemonicMDR pack has been modified on files:

Packs/mnemonicMDR/Integrations/ArgusManagedDefence/ArgusManagedDefence.yml
Please review the changes here

Copy link

github-actions bot commented Jun 5, 2025

Your contributed Arcanna pack has been modified on files:

Packs/Arcanna/Integrations/ArcannaAI/ArcannaAI.yml
Please review the changes here

Copy link

github-actions bot commented Jun 5, 2025

Your contributed CofenseTriage pack has been modified on files:

Packs/CofenseTriage/Integrations/CofenseTriagev3/CofenseTriagev3.yml
Please review the changes here

Copy link

github-actions bot commented Jun 5, 2025

Your contributed Respond pack has been modified on files:

Packs/Respond/Integrations/RespondAnalyst/RespondAnalyst.yml
Please review the changes here

Copy link

github-actions bot commented Jun 5, 2025

Your contributed CyCognito pack has been modified on files:

Packs/CyCognito/Integrations/CyCognito/CyCognito.yml
Please review the changes here

Copy link
Contributor

@ShahafBenYakir ShahafBenYakir left a comment

Choose a reason for hiding this comment

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

looks fine by me

@content-bot
Copy link
Collaborator

Validate summary
The following errors were thrown as a part of this pr: DO105, BC112, DO106, RM108, ST111, RN107.
The following errors can be ignored: RM108.
The following errors cannot be ignored: DO105, BC112, DO106, ST111, RN107.
The following errors don't run as part of the nightly flow and therefore can be force merged: DO105, BC112, DO106, RM108, ST111, RN107.

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

@jlevypaloalto jlevypaloalto added the ready-for-pipeline-running Whether the pr is ready for running the whole pipeline, including testing on SAAS machines label Jun 9, 2025
Copy link
Contributor

@tschanfeld tschanfeld left a comment

Choose a reason for hiding this comment

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

looks fine

@content-bot
Copy link
Collaborator

This PR was automatically updated by a GitHub Action

  • ServiceNow pack version was bumped to 2.7.19.

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs-approved 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.

4 participants