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

Add Rails at scale to the list of blogs #3432

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

vinistock
Copy link

Add Rails at Scale to the list of blogs. It's another good resource for learning Ruby and keeping up with some of the latest developments.

@vinistock vinistock requested a review from a team as a code owner November 26, 2024 21:29
Comment on lines 24 to 26
* [**Rails at Scale**][rails-at-scale] contains posts about the work
being done around both Ruby and Rails by the Ruby and Rails Infrastructure
team at Shopify.
Copy link
Member

Choose a reason for hiding this comment

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

Thank you for your contribution!

The content from the site is absolutely valuable and relevant. However, my opinion is that Ruby official site should maintain neutrality and avoid promoting specific companies. I suggest rephrasing the description to emphasize the content’s relevance to Ruby without referencing the team/company behind it (but definitely appreciate all the work the team/company are doing 💚).

Copy link
Member

Choose a reason for hiding this comment

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

Also should this be under Blogs of Note?

Copy link
Author

Choose a reason for hiding this comment

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

Yeah, that's fair. I tried rephrasing it, let me know what you think.

Also should this be under Blogs of Note?

I moved it there. Is Mining for blogs supposed to be a list of aggregators where you can discover blogs? I guess I didn't understand the title at first glance.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants