-
Notifications
You must be signed in to change notification settings - Fork 40
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 BMC Logo #57
Add BMC Logo #57
Conversation
@shahed-al I found where to put the link |
@xmulligan Is this ready to be merged then? |
@edwarnicke Yes I think it is ready |
Yes it’s ready
…On Wednesday, May 5, 2021, Ed Warnicke ***@***.***> wrote:
@xmulligan <https://github.com/xmulligan> Is this ready to be merged then?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#57 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/APHMGWMHYKF6HXZVYXMBWI3TMGD5DANCNFSM435KJNQA>
.
|
@xmulligan We have a bit of a mechanical problem (see https://deploy-preview-57--inclusivenaming.netlify.app/ )... img src was wrong. I've suggested what I think is the right thing. If so, please correct and we can proceed to merge :) |
Co-authored-by: Ed Warnicke <[email protected]>
Thanks @edwarnicke it works now. appreciate the help |
/lgtm |
@xmulligan We all get by with a little help from our friends... I've made many many many such small errors in my day, and had most of them caught by friends in code review :) Just paying it forward here :) |
No description provided.