Keeping Code Style Sanity in a 13-year-old Codebase

on YouTube ↗23m

" summary

Shopify documents migrating a 13‑year codebase of 700,000 lines of Ruby with ~400 Rails developers and 800+ contributors, using RuboCop plus a custom PR linter Felicia (Politia) that first commented on PRs, then switched to commit-status checks with about 50% adoption. Noisy comments force moving rules into CI, auto‑correcting many offenses via a bot, but linting slows tests so teams exclude huge migrations and plan PR auto‑commits for correctable fixes.

AI-written summary · about

" chapters (7)

1. 0:00Introduction and context
2. 3:09Evolution of code style at Shopify
3. 7:18Introducing RuboCop and HoundCI
4. 11:38Fixing existing violations
5. 16:12Refining tooling and strategy
6. 19:20Future improvements and goals
7. 22:14Final takeaways and conclusion

" related talks

• 24mKeeping Code Style Sanity in a 13-year-old CodebaseGabi Stefanini · THE CONF 2017
• 33m​Keeping Code Style Sanity in a 10-year-old CodebaseGabi Stefanini · RailsConf 2017
• 23mEmbracing Code Quality with RubocopShloka Shah · RubyConf India 2023
• 34mShitlist-driven development and other tricks for working on large codebasesFlorian Weingarten · RedDotRubyConf 2017
• 28mRefactoring for Rails - using deodorant to prevent code smellsRodrigo Urubatan · RubyConf TH 2022
• 31mHow GitHub uses lintersJoel Hawksley · RubyConf 2021
• 33mMeet Shopify's Ruby and Rails Infrastructure Teamrafael franza · RubyConf 2021
• 26mSemi Automatic Code ReviewRichard Huang · RailsConf 2012