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

Analyze your changeset in a branch before git p...

Avatar for Janos Gyerik Janos Gyerik
December 08, 2016
Technology
0
640

Analyze your changeset in a branch before git push using SonarLint + Git hooks; Round #2

Avatar for Janos Gyerik

Janos Gyerik

December 08, 2016
Tweet

More Decks by Janos Gyerik

See All by Janos Gyerik
Capture The Flag at SonarSource 2019
Avatar for Janos Gyerik janosgyerik
1
38
Keyboard shortcuts
Avatar for Janos Gyerik janosgyerik
0
140
Analyze your changeset in a branch before git push using SonarLint + Git hooks
Avatar for Janos Gyerik janosgyerik
0
350
Unwatch GitHub repos
Avatar for Janos Gyerik janosgyerik
0
300
Code Reviews
Avatar for Janos Gyerik janosgyerik
0
150
Playing with Yahoo! Pipes
Avatar for Janos Gyerik janosgyerik
0
250
Cool features of GitHub
Avatar for Janos Gyerik janosgyerik
0
1.3k
Time-saving tricks on the command line
Avatar for Janos Gyerik janosgyerik
1
4.2k

Other Decks in Technology

See All in Technology
伴走から自律へ： 形式知へと導くSREイネーブリングによる プロダクトチームの信頼性オーナーシップ向上 / SRE NEXT 2025
Avatar for Visional Engineering ＆ Design visional_engineering_and_design
3
450
CDKコード品質UP！ナイスな自作コンストラクタを作るための便利インターフェース
Avatar for Haruka Sakihara harukasakihara
2
230
公開初日に Gemini CLI を試した話や FFmpeg と組み合わせてみた話など / Gemini CLI 初学者勉強会（#AI道場）
Avatar for you（@youtoy） you
PRO
0
1.3k
Rethinking Incident Response: Context-Aware AI in Practice
Avatar for rrreeeyyy rrreeeyyy
1
890
Transformerを用いたアイテム間の 相互影響を考慮したレコメンドリスト生成
Avatar for Recruit recruitengineers
PRO
0
180
TROCCO と Stripeを使ったら Tenant-aware Telemetryを有効活用できそう！ / Using TROCCO and Stripe, it seems possible to effectively utilize tenant-aware telemetry!
Avatar for Yuki.Ozasa oztick139
0
110
〜『世界中の家族のこころのインフラ』を目指して”次の10年”へ〜 SREが導いたグローバルサービスの信頼性向上戦略とその舞台裏 / Towards the Next Decade: Enhancing Global Service Reliability
Avatar for kohbis kohbis
3
1.3k
SREのためのeBPF活用ステップアップガイド
Avatar for Sohei Iwahori egmc
2
1.2k
SRE不在の開発チームが障害対応と 向き合った100日間 / 100 days dealing with issues without SREs
Avatar for katsukamaru shin1988
2
2k
サービスを止めるな！ DDoS攻撃へのスマートな備えと最前線の事例
Avatar for coconala_engineer coconala_engineer
1
140
shake-upを科学する
Avatar for Jack rsakata
7
990
60以上のプロダクトを持つ組織における開発者体験向上への取り組み - チームAPIとBackstageで構築する組織の可視化基盤 - / sre next 2025 Efforts to Improve Developer Experience in an Organization with Over 60 Products
Avatar for VTRyo vtryo
3
1.8k

Featured

See All Featured
The Art of Delivering Value - GDevCon NA Keynote
Avatar for David Neal reverentgeek
15
1.6k
The MySQL Ecosystem @ GitHub 2015
Avatar for Sam Lambert samlambert
251
13k
Improving Core Web Vitals using Speculation Rules API
Avatar for Sergey Chernyshev sergeychernyshev
18
990
Designing Dashboards & Data Visualisations in Web Apps
Avatar for Des Traynor destraynor
231
53k
The Web Performance Landscape in 2024 [PerfNow 2024]
Avatar for Tammy Everts tammyeverts
8
700
Evolution of real-time – Irina Nazarova, EuRuKo, 2024
Avatar for Irina Nazarova irinanazarova
8
830
RailsConf & Balkan Ruby 2019: The Past, Present, and Future of Rails at GitHub
Avatar for Eileen M. Uchitelle eileencodes
138
34k
StorybookのUI Testing Handbookを読んだ
Avatar for Shingo Yamazaki zakiyama
30
5.9k
Why You Should Never Use an ORM
Avatar for John Nunemaker jnunemaker
PRO
58
9.5k
Code Review Best Practice
Avatar for Trisha Gee trishagee
69
19k
Code Reviewing Like a Champion
Avatar for maltzj maltzj
524
40k
Performance Is Good for Brains [We Love Speed 2024]
Avatar for Tammy Everts tammyeverts
10
970

Transcript

  1. ANALYZE YOUR CHANGESET BEFORE GIT PUSH WITH SONARLINT + GIT

    HOOKS ROUND #2 https://speakerdeck.com/janosgyerik/sonarlint-before-git-push-2

  2. Recap

  3. Motivation * Changeset = branch = many commits * Automatic

    (+bypass)

  4. git clone https://github.com/janosgyerik/sonarlint-git-hooks cd sonarlint-git-hooks ./get-and-setup-sonarlint.sh hooks=$PWD cd /path/to/your/project $hooks/install.sh

    Setup

  5. Difference from sonar-github * Runs locally * Fast feedback *

    No-brainer setup

  6. Difference from pre-commit checks in IntelliJ * Many commits vs

    one * Less frequent * IntelliJ not needed

  7. “custom” sonarlint? custom == future

  8. None

  9. Tip #1 : bypass the analysis SKIPSONARLINT=1 git push -u

    origin HEAD

  10. Tip #2 : git alias to bypass the analysis somename

    = "!f() { SKIPSONARLINT=1 git push -u origin HEAD \"$@\"; }; f"

  11. Tip #3 : enable pre-commit hook mv .git/hooks/pre-commit.optional .git/hooks/pre-commit

  12. https://github.com/janosgyerik/sonarlint-git-hooks/issues Problems? Ideas? Feature requests?