-
Notifications
You must be signed in to change notification settings - Fork 171
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
Fix the example-cats-config #451
Conversation
We have created an issue in Pivotal Tracker to manage this: https://www.pivotaltracker.com/story/show/176348431 The labels on this github issue will be updated when the story is started. |
|
hey @ameowlia, could you please make sure you're properly associated with the commit? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These changes look good to me - thank you!
2bc3757
to
5ec06d9
Compare
We have created an issue in Pivotal Tracker to manage this: https://www.pivotaltracker.com/story/show/176364844 The labels on this github issue will be updated when the story is started. |
I fixed the commit and I closed+reopened in hopes that the EasyCLA would run again 🤔 Yesterday I had this same problem with this PR, but it seems to be fixed now. I wonder why not here. |
add required property credhub_secret to config Co-authored-by: George Gelashvili <ggelashvili@vmware.com>
5ec06d9
to
af5b5a7
Compare
I moved it from authored by george to authored by me and it seems to have fixed it. I wonder if everyone has to resign like I did yesterday? |
What is this change about?
Update the example config to add required property "credhub_secret".