content icon indicating copy to clipboard operation
content copied to clipboard

[AzureRiskyUsers] New commands

Open jlevypaloalto opened this issue 2 months ago • 4 comments

Status

  • [x] 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

jlevypaloalto avatar Nov 09 '25 16:11 jlevypaloalto

Coverage

Coverage Report
FileStmtsMissCoverMissing
Packs/AzureRiskyUsers/Integrations/AzureRiskyUsers
   AzureRiskyUsers.py2053781%32–35, 115, 180, 187–188, 199, 206–207, 335–336, 392, 400–402, 404–405, 414–415, 421, 539–540, 544–545, 552–553, 666–668, 683, 708, 723–724, 728–729
TOTAL2053781% 

Tests Skipped Failures Errors Time
21 0 :zzz: 0 :x: 0 :fire: 2.517s :stopwatch:

github-actions[bot] avatar Nov 09 '25 16:11 github-actions[bot]

Nice! Didn't we agree to add the failures to the context as well?

It was suggested, but they didn't want it.

jlevypaloalto avatar Nov 25 '25 07:11 jlevypaloalto

@AradCarmi docs approved, see comments

talihaff avatar Nov 27 '25 12:11 talihaff

Validate summary The following errors were thrown as a part of this pr: . If the AG100 validation in the pre-commit GitHub Action fails, the pull request cannot be force-merged.

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

content-bot avatar Dec 05 '25 11:12 content-bot