Skip to content

Conversation

@sameersaeed
Copy link

Adds tests for hook injection validation per #86

cc @AbdelrahmanElawady, @mikebrow - feel free to let me know if you have any suggestions!

@sameersaeed sameersaeed force-pushed the hook-injector-tests branch 3 times, most recently from 5c68a54 to eae8638 Compare July 25, 2025 17:59
@sameersaeed sameersaeed marked this pull request as draft October 28, 2025 16:24
@mikebrow
Copy link
Member

mikebrow commented Nov 3, 2025

fell off my radar... can we get a rebase?

@sameersaeed
Copy link
Author

@mikebrow - I've rebased on main, also made a minor change so that the hook runs on container start rather than on pre-start

Feel free to let me know if there's anything I can improve!

@sameersaeed sameersaeed force-pushed the hook-injector-tests branch 2 times, most recently from e254ad8 to 87ef644 Compare January 13, 2026 03:02
Signed-off-by: Sameer <sameer.saeed@live.ca>
@sameersaeed
Copy link
Author

Rebased on latest main, can also help look into adding critest tests for this if needed

@sameersaeed sameersaeed marked this pull request as ready for review January 13, 2026 03:12
Copy link
Member

@mikebrow mikebrow left a comment

Choose a reason for hiding this comment

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

LGTM

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.

2 participants