Skip to content
This repository has been archived by the owner on May 7, 2024. It is now read-only.

update firewall summary table #403

Merged
merged 3 commits into from
Mar 26, 2024
Merged

update firewall summary table #403

merged 3 commits into from
Mar 26, 2024

Conversation

ejhenry
Copy link
Contributor

@ejhenry ejhenry commented Mar 26, 2024

Overview/Summary

Update firewall summary table with recently added recommendations

Related Issues/Work Items

None

This PR fixes/adds/changes/removes

  1. Update fw summary table

Breaking Changes

  1. None

As part of this Pull Request I have

  • Read the Contribution Guide and ensured this PR is compliant with the guide
  • Checked for duplicate Pull Requests
  • Associated it with relevant GitHub Issues or ADO Work Items (Internal Only)
  • Ensured my code/branch is up-to-date with the latest changes in the main branch
  • Ensured PR tests are passing
  • Performed testing and provided evidence (e.g. screenshot of output) for any changes associated to ARG queries and/or scripts
  • Updated relevant and associated documentation (e.g. Contribution Guide, Docs etc.)

@ejhenry ejhenry requested review from a team as code owners March 26, 2024 22:30
DaFitRobsta
DaFitRobsta approved these changes Mar 26, 2024
View reviewed changes
Copy link
Contributor

@DaFitRobsta DaFitRobsta left a comment

Choose a reason for hiding this comment

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

LGTM

@ejhenry ejhenry merged commit 20d4b5d into Azure:main Mar 26, 2024
4 checks passed
@ejhenry ejhenry deleted the fw-summary branch March 26, 2024 23:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@DaFitRobsta DaFitRobsta DaFitRobsta approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ejhenry @DaFitRobsta