Skip to content

fix: omit empty credentials#384

Open
tim-goto wants to merge 6 commits intojfrog:mainfrom
tim-goto:feature/tharing/fix-empty-credentials
Open

fix: omit empty credentials#384
tim-goto wants to merge 6 commits intojfrog:mainfrom
tim-goto:feature/tharing/fix-empty-credentials

Conversation

@tim-goto
Copy link
Copy Markdown

@tim-goto tim-goto commented Mar 5, 2026

  • All tests passed. If this feature is not already covered by the tests, I added new tests.
  • All static analysis checks passed.
  • Appropriate label is added to auto generate release notes.
  • I used gofmt for formatting the code before submitting the pull request.
  • PR description is clear and concise, and it includes the proposed solution/fix.

@tim-goto
Copy link
Copy Markdown
Author

tim-goto commented Mar 6, 2026

@naveenku-jfrog can you please take a look here? our TSM contact is Douglas Hemsworth.

@naveenku-jfrog naveenku-jfrog added the safe to test Approve running integration tests on a pull request label Mar 10, 2026
@github-actions github-actions bot removed the safe to test Approve running integration tests on a pull request label Mar 10, 2026
@naveenku-jfrog naveenku-jfrog added bug Something isn't working safe to test Approve running integration tests on a pull request labels Mar 10, 2026
@github-actions github-actions bot removed the safe to test Approve running integration tests on a pull request label Mar 10, 2026
@naveenku-jfrog naveenku-jfrog added safe to test Approve running integration tests on a pull request and removed safe to test Approve running integration tests on a pull request labels Mar 10, 2026
@github-actions github-actions bot removed the safe to test Approve running integration tests on a pull request label Mar 10, 2026
Copy link
Copy Markdown
Collaborator

@naveenku-jfrog naveenku-jfrog left a comment

Choose a reason for hiding this comment

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

LGTM

@naveenku-jfrog naveenku-jfrog requested a review from bhanurp March 10, 2026 08:52
@github-actions
Copy link
Copy Markdown
Contributor

👍 Frogbot scanned this pull request and did not find any new security issues.


Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants