Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor(RedisInstance): move auth secret to separate file #886

Merged

Conversation

dushanpantic
Copy link
Contributor

@dushanpantic dushanpantic commented Dec 12, 2024

Description

Changes proposed in this pull request:

  • ...
  • ...
  • ...

Related issue(s)
#881

@dushanpantic dushanpantic requested a review from a team as a code owner December 12, 2024 11:49
@kyma-bot kyma-bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Dec 12, 2024
Copy link
Contributor

@ijovovic ijovovic left a comment

Choose a reason for hiding this comment

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

lgtm

@kyma-bot kyma-bot added the lgtm Looks good to me! label Dec 12, 2024
@dushanpantic dushanpantic enabled auto-merge (squash) December 12, 2024 11:54
@dushanpantic dushanpantic force-pushed the refactor-auth-secret-location branch from cbc8e1b to ba01e81 Compare December 12, 2024 11:59
@kyma-bot kyma-bot removed the lgtm Looks good to me! label Dec 12, 2024
@kyma-bot
Copy link
Contributor

New changes are detected. LGTM label has been removed.

@kyma-bot kyma-bot added the cla: yes Indicates the PR's author has signed the CLA. label Dec 12, 2024
@dushanpantic dushanpantic merged commit df75b80 into kyma-project:main Dec 12, 2024
15 checks passed
@dushanpantic dushanpantic deleted the refactor-auth-secret-location branch December 12, 2024 12:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes Indicates the PR's author has signed the CLA. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants