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.

Sign up for GitHub

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

Jump to bottom

Add persistToPropertiesFile overload #784

Merged
merged 1 commit into from
Feb 18, 2024
Merged

Add persistToPropertiesFile overload #784

merged 1 commit into from
Feb 18, 2024

Conversation

Or-Geva
Copy link
Collaborator

@Or-Geva Or-Geva commented Feb 15, 2024 •
edited
Loading

  • All tests passed. If this feature is not already covered by the tests, I added new tests.

Add persistToPropertiesFile overload to enable external stream.
Relate to:

@Or-Geva Or-Geva added the improvement Automatically generated release notes label Feb 15, 2024
@Or-Geva Or-Geva requested a review from yahavi February 15, 2024 14:04
@Or-Geva Or-Geva mentioned this pull request Feb 15, 2024
Merged
3 tasks
@Or-Geva Or-Geva added the safe to test Approve running integration tests on a pull request label Feb 15, 2024
@github-actions github-actions bot removed the safe to test Approve running integration tests on a pull request label Feb 15, 2024
@yahavi yahavi added bug Create a report to help us improve and removed improvement Automatically generated release notes labels Feb 17, 2024
@Or-Geva Or-Geva merged commit 338a59e into jfrog:master Feb 18, 2024
51 of 52 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yahavi yahavi yahavi approved these changes

Assignees
No one assigned
Labels
bug Create a report to help us improve
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Or-Geva @yahavi