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

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

Sponsored · Your Podcast. Everywhere. Effortlessly. Share. Educate. Inspire. Entertain. You do you. We'll handle the rest. →
Avatar for Janos Gyerik Janos Gyerik
December 08, 2016
Technology
650
0

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

Avatar for Janos Gyerik

Janos Gyerik

December 08, 2016

More Decks by Janos Gyerik

See All by Janos Gyerik
Capture The Flag at SonarSource 2019
Avatar for Janos Gyerik janosgyerik
1
44
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
370
Unwatch GitHub repos
Avatar for Janos Gyerik janosgyerik
0
320
Code Reviews
Avatar for Janos Gyerik janosgyerik
0
160
Playing with Yahoo! Pipes
Avatar for Janos Gyerik janosgyerik
0
270
Cool features of GitHub
Avatar for Janos Gyerik janosgyerik
0
1.4k
Time-saving tricks on the command line
Avatar for Janos Gyerik janosgyerik
1
4.3k

Other Decks in Technology

See All in Technology
雑談は、センサーだった
Avatar for 株式会社ビットキー / Bitkey Inc. bitkey
PRO
2
220
自動テストだけで
リリース判断できるチームへ - 鍵はテストの量ではなくリリース判断基準の再設計にあった / Redesigning Release Criteria for Lightweight Releases
Avatar for えわ ewa
7
3.5k
AIの揺らぎに“コシ”を与える階層化品質設計
Avatar for ICKX ickx
0
270
サービスの信頼性を高めるため、形骸化した「プロダクションミーティング」を立て直すまでの取り組み
Avatar for すてにゃん stefafafan
1
250
Swift Sequence の便利 API 再発見
Avatar for treastrain / Tanaka Ryoga treastrain
1
150
"うちにはまだ早い"は本当？ ─ 小さく始めるPlatform Engineering入門
Avatar for Haruka Sakihara harukasakihara
2
250
ボトムアップ限界を越える - 20チームを束る "Drive Map" / Beyond Bottom-Up: A 'Drive Map' for 20 Teams
Avatar for 株式会社カオナビ kaonavi
0
150
Every Conversation Counts
Avatar for Yasunobu Kawaguchi kawaguti
PRO
0
140
知ってた？JavaScriptの"正しさ"を検証するテストが5万以上もあること(Test262)
Avatar for Riya Amemiya riyaamemiya
1
160
アプリブロック機能のつくりかたと、AIとHTMLの不合理な相性の良さについて
Avatar for kumamotone kumamotone
0
150
「QA＝テスト」「シフトレフト＝スクラムイベントの参加者の一員」の呪縛を解く。アジャイルな開発を止めないために、10Xで挑んだ「右側のしわ寄せ」解消記 #scrumniigata
Avatar for nihonbuson nihonbuson
PRO
3
930
ボトムアップの改善の火を灯し続けろ！〜支援現場で学んだ、消えないための3つの打ち手〜 / 20260509 Kazuki Mori
Avatar for SHIFT EVOLVE shift_evolve
PRO
2
590

Featured

See All Featured
AI Search:
Where Are We & What Can We Do About It?
Avatar for Aleyda Solis aleyda
0
7.4k
We Have a Design System, Now What?
Avatar for Morgane Peng morganepeng
55
8.1k
Easily Structure & Communicate Ideas using Wireframe
Avatar for Afnizar Nur Ghifari afnizarnur
194
17k
Are puppies a ranking factor?
Avatar for Jono Alderson jonoalderson
1
3.4k
Designing Experiences People Love
Avatar for Jonathan Moore moore
143
24k
Raft: Consensus for Rubyists
Avatar for Patrick Van Stee vanstee
141
7.4k
Large-scale JavaScript Application Architecture
Avatar for Addy Osmani addyosmani
515
110k
コードの90%をAIが書く世界で何が待っているのか / What awaits us in a world where 90% of the code is written by AI
Avatar for r-kagaya rkaga
61
43k
GitHub's CSS Performance
Avatar for Jon Rohan jonrohan
1032
470k
Music & Morning Musume
Avatar for Bryan Veloso bryan
47
7.2k
Un-Boring Meetings
Avatar for Sebastian Deterding codingconduct
0
280
Templates, Plugins, & Blocks: Oh My! Creating the theme that thinks of everything
Avatar for Michelle Schulp Hunt marktimemedia
31
2.8k

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?