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

fix(injector): fix issue nil interface conversion #141

Merged
merged 2 commits into from
Jan 30, 2024
Merged

fix(injector): fix issue nil interface conversion #141

merged 2 commits into from
Jan 30, 2024

Conversation

csatib02
Copy link
Member

@csatib02 csatib02 commented Jan 24, 2024

Overview

Fixes: Issue #-81 in Vault-env repo.

@csatib02 csatib02 requested a review from a team as a code owner January 24, 2024 13:26
@csatib02 csatib02 requested review from akijakya and removed request for a team January 24, 2024 13:26
@github-actions github-actions bot added the size/S Denotes a PR that changes 10-99 lines label Jan 24, 2024
Copy link
Member

@akijakya akijakya left a comment

Choose a reason for hiding this comment

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

LGTM, nice work!

@akijakya akijakya requested a review from ramizpolic January 25, 2024 16:20
injector/injector.go Outdated Show resolved Hide resolved
Signed-off-by: Bence Csati <[email protected]>
@csatib02 csatib02 requested a review from ramizpolic January 30, 2024 13:44
@ramizpolic ramizpolic merged commit 67fc632 into bank-vaults:main Jan 30, 2024
12 checks passed
@csatib02 csatib02 deleted the fix/error-nil-conversion branch February 12, 2024 08:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
size/S Denotes a PR that changes 10-99 lines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants