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

[chore|helmchart-v0.1.2] vulnerability,helm-lint &backend condition fix. #28

Merged
merged 1 commit into from
Dec 4, 2023

Conversation

adityagajbhiye9
Copy link
Contributor

@adityagajbhiye9 adityagajbhiye9 commented Dec 4, 2023

Description

  • helm lint workflow fix for dependency.
  • update changelog and docs

Pre-review checks

Please ensure to do as many of the following checks as possible, before asking for committer review:

…ected.

* changed to v2.3.2 docker image version.
* veracode vulnerability fix in AppV- 2.3.2.
* updated backend-deployment.yaml for condition check.
* helm lint workflow fix for dependency.
@adityagajbhiye9
Copy link
Contributor Author

adityagajbhiye9 commented Dec 4, 2023

@FaGru3n I have corrected the workflow. Please review and approve🤞
and let us know if any required changes to be done.

Copy link
Contributor

@FaGru3n FaGru3n left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link

@hzierer hzierer left a comment

Choose a reason for hiding this comment

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

LGTM

@FaGru3n
Copy link
Contributor

FaGru3n commented Dec 4, 2023

@adityagajbhiye9 please make sure that the pr decriptions fits to your changes.

@FaGru3n FaGru3n merged commit 71f7b7d into eclipse-tractusx:main Dec 4, 2023
3 checks passed
@adityagajbhiye9 adityagajbhiye9 deleted the v2.3.2 branch December 4, 2023 10:53
@adityagajbhiye9 adityagajbhiye9 restored the v2.3.2 branch December 4, 2023 10:53
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.

3 participants