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

Add external dependencies rewrite parameters for helm remote repos #673

Merged
merged 1 commit into from Dec 7, 2022

Conversation

fabianrindlisbacher
Copy link
Contributor

  • All tests passed. If this feature is not already covered by the tests, I added new tests.
  • All static analysis checks passed.
  • This pull request is on the dev branch.
  • I used gofmt for formatting the code before submitting the pull request.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 3, 2022

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@fabianrindlisbacher
Copy link
Contributor Author

I have read the CLA Document and I hereby sign the CLA

@fabianrindlisbacher
Copy link
Contributor Author

fabianrindlisbacher commented Nov 3, 2022

The changes are needed to implement the new dependency rewrite functionality describe here: https://www.jfrog.com/confluence/display/JFROG/Kubernetes+Helm+Chart+Repositories -> Under Remote Repositories

Would be great if these changes would be release asap, since we've got alot of our devs waiting for a feature that requires this flag.

@fabianrindlisbacher
Copy link
Contributor Author

@eyalbe4 @sverdlov93 can you review these changes?

@yahavi yahavi added the safe to test Approve running integration tests on a pull request label Dec 7, 2022
@github-actions github-actions bot removed the safe to test Approve running integration tests on a pull request label Dec 7, 2022
Copy link
Member

@yahavi yahavi left a comment

Choose a reason for hiding this comment

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

Thanks for this contribution, @fabianrindlisbacher!

@yahavi yahavi added the improvement Automatically generated release notes label Dec 7, 2022
@yahavi yahavi merged commit 10dc26c into jfrog:dev Dec 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement Automatically generated release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants