-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Ciac 604 pr number on rn #19074
Ciac 604 pr number on rn #19074
Conversation
3ff9025
to
c355344
Compare
Your contributed Wiz pack has been modified on files:Packs/Wiz/Integrations/Wiz/Wiz.py |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work.
- Why haven't we tested the get_version_to_pr_numbers function? Let's add a test for it.
- For the future when you are running tests/experiments make sure not to do it on partner packs as it alerts them (see above) that their pack is being modified, let's change the test to another pack.
34f09b8
to
dbbf945
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As discussed, please open the issue regarding the commit pattern
mocker.patch("os.listdir", return_value=dir_list) | ||
mocker.patch("os.path.exists", return_value=True) | ||
|
||
class GitMock: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why aren't we uniting this with the other GitMock class?
f7f9855
to
5f57791
Compare
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
fixes: link to the issue
Description
A few sentences describing the overall goals of the pull request's commits.
Screenshots
Paste here any images that will help the reviewer
Minimum version of Cortex XSOAR
Does it break backward compatibility?
Must have