Upgrade to Pro — share decks privately, control downloads, hide ads and more …

Saddler - HCMC meetup 2015

Sponsored · SiteGround - Reliable hosting with speed, security, and support you can count on. →
Avatar for Vinh Nguyen Le Vinh Nguyen Le
November 25, 2015
Technology
0
300

Saddler - HCMC meetup 2015

5 minutes introduce and setup saddler for code review

Avatar for Vinh Nguyen Le

Vinh Nguyen Le

November 25, 2015
Tweet

More Decks by Vinh Nguyen Le

See All by Vinh Nguyen Le
Rails girls - Part 1
Avatar for Vinh Nguyen Le vinhngl
0
72
Introduction Heroku
Avatar for Vinh Nguyen Le vinhngl
0
140

Other Decks in Technology

See All in Technology
予期せぬコストの急増を障害のように扱う――「コスト版ポストモーテム」の導入とその後の改善
Avatar for Masahiro Yoshizawa muziyoshiz
1
2k
We Built for Predictability; The Workloads Didn’t Care
Avatar for Michael Stahnke stahnma
0
140
20260204_Midosuji_Tech
Avatar for Takuya Yonezawa takuyay0ne
1
160
Kiro IDEのドキュメントを全部読んだので地味だけどちょっと嬉しい機能を紹介する
Avatar for khmoryz khmoryz
0
210
Codex 5.3 と Opus 4.6 にコーポレートサイトを作らせてみた / Codex 5.3 vs Opus 4.6
Avatar for ama-ch ama_ch
0
190
仕様書駆動AI開発の実践： Issue→Skill→PRテンプレで 再現性を作る
Avatar for 西岡 賢一郎 (Kenichiro Nishioka) knishioka
2
680
ClickHouseはどのように大規模データを活用したAIエージェントを全社展開しているのか
Avatar for Miki Matsumoto mikimatsumoto
0
260
Greatest Disaster Hits in Web Performance
Avatar for Estela Franco guaca
0
280
Webhook best practices for rock solid and resilient deployments
Avatar for Guillaume Laforge glaforge
2
300
GitHub Issue Templates + Coding Agentで簡単みんなでIaC/Easy IaC for Everyone with GitHub Issue Templates + Coding Agent
Avatar for AEON aeonpeople
1
260
SREのプラクティスを用いた3領域同時 マネジメントへの挑戦 〜SRE・情シス・セキュリティを統合した チーム運営術〜
Avatar for coconala_engineer coconala_engineer
2
730
レガシー共有バッチ基盤への挑戦 - SREドリブンなリアーキテクチャリングの取り組み
Avatar for Konishi tatsuhiro tatsukoni
0
220

Featured

See All Featured
Hiding What from Whom? A Critical Review of the History of Programming languages for Music
Avatar for Tomoya Matsuura tomoyanonymous
2
420
Jess Joyce - The Pitfalls of Following Frameworks
Avatar for Tech SEO Connect techseoconnect
PRO
1
67
Designing Dashboards & Data Visualisations in Web Apps
Avatar for Des Traynor destraynor
231
54k
The Web Performance Landscape in 2024 [PerfNow 2024]
Avatar for Tammy Everts tammyeverts
12
1k
Six Lessons from altMBA
Avatar for Skipper Chong Warson skipperchong
29
4.2k
Color Theory Basics | Prateek | Gurzu
Avatar for Gurzu gurzu
0
200
4 Signs Your Business is Dying
Avatar for Josh Pigford shpigford
187
22k
CSS Pre-Processors: Stylus, Less & Sass
Avatar for Bermon Painter bermonpainter
359
30k
Avoiding the “Bad Training, Faster” Trap in the Age of AI
Avatar for Mike Taylor tmiket
0
79
AI Search:
Where Are We & What Can We Do About It?
Avatar for Aleyda Solis aleyda
0
7k
Agile Leadership in an Agile Organization
Avatar for Dr. Kim W Petersen kimpetersen
PRO
0
83
Stewardship and Sustainability of Urban and Community Forests
Avatar for Eric Wiseman pwiseman
0
110

Transcript

  1. Check Style to anywhere Introduce Saddler Presenter: vinhnglx

  2. Code Review

  3. General Practices • Naming Convention • Indentation • Brace style

    • Commenting • Code consistency

  4. Code Metrics • RuboCop • Rails best practices • Reek

    • Hound CI - 12$ / month

  5. RuboCop Rails best practice Reek Saddler Reporters GitHub Pull Requests

    CI Services

  6. • GITHUB_ACCESS_TOKEN • gem ‘saddler’ & ‘saddler-reporter-github’ • metrics gems

    (rubocop - rails_best_practices …)
 Requirements

  7. • https://transfer.sh/EgzcG/circle.yml • https://transfer.sh/Infbv/saddler.sh • https://github.com/vinhnglx/twitter-trends/pull/1 Setup & Result

  8. THANK YOU