Skip to content
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

Update 'rails_welcome.png' to reflect a more diverse population #32735

Merged
merged 2 commits into from Apr 26, 2018

Conversation

@asianmack
Copy link
Contributor

@asianmack asianmack commented Apr 26, 2018

rails-welcome-new

cc @dhh

@dhh dhh merged commit 46b46e8 into rails:master Apr 26, 2018
1 of 2 checks passed
1 of 2 checks passed
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
codeclimate All good!
Details
@davetapley
Copy link

@davetapley davetapley commented Apr 26, 2018

Love it! Probably can update the Twitter banner too? 💫

@dhh
Copy link
Member

@dhh dhh commented Apr 26, 2018

Good point. I've updated the banner 👍

@coderliu
Copy link

@coderliu coderliu commented Apr 27, 2018

Where is the dog? This hurts people who love dog.

rafaelfranca added a commit that referenced this pull request Apr 27, 2018
* Update 'rails_welcome.png' to reflect a more diverse population

* Cleanup 'rails_welcome.png'
@ismailarilik
Copy link

@ismailarilik ismailarilik commented Apr 29, 2018

Cat stays but dog leaves; it is not inclusive I think (Let's remove all animals from this image to get more place for people.). Also I think inclusive should mean including members of common religions and races.

@StanBright
Copy link

@StanBright StanBright commented May 2, 2018

Yeah, I noticed too that the dog is gone, but the cat is still present. In the end of the day - what's an app on the web without cats!?

@onomojo
Copy link

@onomojo onomojo commented May 2, 2018

There's no dog anymore. I'm offended.

@rails rails locked and limited conversation to collaborators May 2, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

7 participants