Skip to content

Add k8s black lives matter statement #4022

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
Merged
merged 2 commits into from
Oct 13, 2020
Merged

Add k8s black lives matter statement #4022

merged 2 commits into from
Oct 13, 2020

Conversation

matthewcarleton
Copy link
Contributor

fixes #3975

@asmacdo asmacdo closed this Oct 13, 2020
@asmacdo asmacdo reopened this Oct 13, 2020
camilamacedo86
camilamacedo86 reviewed Oct 13, 2020
View reviewed changes
@asmacdo asmacdo changed the title updates Add k8s black lives matter statement Oct 13, 2020
@asmacdo
Copy link
Member

asmacdo commented Oct 13, 2020

Looks good on my phone too

Screenshot_20201013-103432

@asmacdo
Copy link
Member

asmacdo commented Oct 13, 2020

I checked it at many widths, they all look great.

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Oct 13, 2020
camilamacedo86
camilamacedo86 approved these changes Oct 13, 2020
View reviewed changes
Copy link
Contributor

@camilamacedo86 camilamacedo86 left a comment

Choose a reason for hiding this comment

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

/lgtm

@asmacdo @joelanford
Update website/layouts/index.html
829dbab 
Co-authored-by: Joe Lanford <joe.lanford@gmail.com>
@openshift-ci-robot openshift-ci-robot removed the lgtm Indicates that a PR is ready to be merged. label Oct 13, 2020
@jmrodri
Copy link
Member

jmrodri commented Oct 13, 2020

Looks fine in Firefox on Linux:

  1. Full screen
    image

  2. odd size
    image

  3. tiny browser
    image

Checked on Android 11 with Firefox and Chrome, looks like the screenshot @asmacdo shared in a previous comment.

jmrodri
jmrodri approved these changes Oct 13, 2020
View reviewed changes
Copy link
Member

@jmrodri jmrodri left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Oct 13, 2020
@asmacdo asmacdo merged commit b3125be into operator-framework:master Oct 13, 2020
estroz pushed a commit to estroz/operator-sdk that referenced this pull request Oct 14, 2020
@matthewcarleton @asmacdo
@joelanford @estroz
Add k8s black lives matter statement (operator-framework#4022)
e18d24f 
* updates

* Update website/layouts/index.html

Co-authored-by: Joe Lanford <joe.lanford@gmail.com>

Co-authored-by: matthew carleton <matthewcarleton@matthews-MacBook-Pro.local>
Co-authored-by: Austin Macdonald <austin@redhat.com>
Co-authored-by: Joe Lanford <joe.lanford@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@joelanford joelanford joelanford left review comments

@jmrodri jmrodri jmrodri approved these changes

@camilamacedo86 camilamacedo86 camilamacedo86 approved these changes

@theishshah theishshah Awaiting requested review from theishshah

+1 more reviewer

@asmacdo asmacdo asmacdo left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

@jmrodri jmrodri

@asmacdo asmacdo

@camilamacedo86 camilamacedo86

Labels

lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Re-add Black Lives Matter banner (without Red Hat branding)

6 participants

@matthewcarleton @asmacdo @jmrodri @joelanford @camilamacedo86 @openshift-ci-robot