Skip to content
This repository has been archived by the owner on May 3, 2024. It is now read-only.

Removed test credentials from pulsar-env.sh #1667

Merged
merged 1 commit into from
Dec 7, 2023
Merged

Removed test credentials from pulsar-env.sh #1667

merged 1 commit into from
Dec 7, 2023

Conversation

shailesh-vaidya
Copy link
Contributor

@shailesh-vaidya shailesh-vaidya commented Dec 7, 2023 •
edited
Loading

Describe your changes in brief

Removed test credentials from the codebase.

Changes

  • Why is this change required? What problem does it solve?
    Remove sensitive information from codebase.
  • If proposing a new change then please raise an issue first

How Has This Been Tested? (Optional)

  • Please describe in detail how you tested your changes.
  • Include details of your testing environment, and the tests you ran to
  • How your change affects other areas of the code, etc.

Screenshots (if appropriate)

Checklist

  • tested locally
  • added new dependencies
  • updated the docs
  • added a test

@cla-bot cla-bot bot added the cla-signed label Dec 7, 2023
@shailesh-vaidya shailesh-vaidya marked this pull request as ready for review December 7, 2023 10:04
@shailesh-vaidya shailesh-vaidya self-assigned this Dec 7, 2023
pankajgborole
pankajgborole approved these changes Dec 7, 2023
View reviewed changes
Copy link
Contributor

@pankajgborole pankajgborole left a comment

Choose a reason for hiding this comment

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

Approve

@shailesh-vaidya shailesh-vaidya merged commit b6832b6 into Seagate:main Dec 7, 2023
4 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@pankajgborole pankajgborole pankajgborole approved these changes

@gauravchaudhari02 gauravchaudhari02 gauravchaudhari02 approved these changes

Assignees

@shailesh-vaidya shailesh-vaidya

Labels
cla-signed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@shailesh-vaidya @gauravchaudhari02 @pankajgborole