Conversation
🤖 AI-Powered Code Review AvailableYou can leverage AI-powered code review to assist with this PR! Available Commands:
|
|
🤖 Analysis started. Please wait for results... |
🤖 AI Review DisclaimerThis 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. |
There was a problem hiding this comment.
Hi! Thanks for your contribution to the Azure pack.
@julieschwartz18, @RotemAmit please review and approve the results generated by the AI Reviewer by responding 👍 on this comment.
|
🤖 Analysis started. Please wait for results... |
🤖 AI Review DisclaimerThis 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. |
There was a problem hiding this comment.
🔄 Re-reviewing updates...
Important
To maintain review integrity, threads marked as Resolved without accompanying changes or explanations will be automatically unresolved for further verification.
Hi there! Thanks for contributing to the Azure integration. I've left a few notes on your changes, mostly regarding a copy-paste error where Server Name is incorrectly mapped to resourceGroup. fix a couple of minor YAML formatting issues like indentation and capitalizing String types. Thanks for your work on this!
@RotemAmit, @julieschwartz18, @marketplace-ai-reviewer please review and approve the results generated by the AI Reviewer by responding 👍 on this comment.
julieschwartz18
left a comment
There was a problem hiding this comment.
@ilaredo Doc review complete, please check the comments. In the meantime adding the label docs-approved
julieschwartz18
left a comment
There was a problem hiding this comment.
@ilaredo Doc review complete, please check the comments. In the meantime adding the label docs-approved
|
Validate summary Verdict: PR can be force merged from validate perspective? ✅ |
Co-authored-by: julieschwartz18 <91824591+julieschwartz18@users.noreply.github.com>
Co-authored-by: julieschwartz18 <91824591+julieschwartz18@users.noreply.github.com>
Co-authored-by: julieschwartz18 <91824591+julieschwartz18@users.noreply.github.com>
Co-authored-by: julieschwartz18 <91824591+julieschwartz18@users.noreply.github.com>
Co-authored-by: julieschwartz18 <91824591+julieschwartz18@users.noreply.github.com>
Co-authored-by: julieschwartz18 <91824591+julieschwartz18@users.noreply.github.com>
|
|
RotemAmit
left a comment
There was a problem hiding this comment.
Nice!
See my comments.
Co-authored-by: RotemAmit <ramit@paloaltonetworks.com>
…-fix-final-v100 # Conflicts: # Packs/Azure/ReleaseNotes/1_1_9.md
RotemAmit
left a comment
There was a problem hiding this comment.
Nice work!
Please update the Azure pack version to 1.1.10, and rename the RN accordingly.
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
Related Issues
CRTX-218061
Description
Must have