Skip to content

Conversation

CasLubbers
Copy link
Contributor

@CasLubbers CasLubbers commented Aug 21, 2025

📌 Summary

Added a wiz install step to our integration.yml

🔍 Reviewer Notes

🧹 Checklist

  • Code is readable, maintainable, and robust.
  • Unit tests added/updated

Copy link
Contributor

@merll merll left a comment

Choose a reason for hiding this comment

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

This seems to include code which is not needed.

Copy link
Collaborator

@ferruhcihan ferruhcihan left a comment

Choose a reason for hiding this comment

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

I couldn't test it with a cluster or Wiz setup but LGTM from my side, aside from the comments mentioned by @merll.

@CasLubbers CasLubbers requested a review from merll August 25, 2025 11:48
Copy link
Contributor

@merll merll left a comment

Choose a reason for hiding this comment

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

Looks good now!

@ElderMatt ElderMatt assigned ElderMatt and unassigned ElderMatt Aug 25, 2025
@merll merll merged commit f3ee976 into main Sep 4, 2025
11 checks passed
@merll merll deleted the APL-1045 branch September 4, 2025 08:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants