loading...

re: Danger on Rails: make robots do some code review for you! VIEW POST

FULL DISCUSSION
 

Kinda of automatic code (or project) checker plugged into a tool to do automatic code (or project) checks ? That's awesome !

With my team, we just write Git pre-hook scripts, why you don't do that too ? There is no interest to push incomplete things on the server no ?

 

Danger allows us to add code review specific checks, for example:

  • PRs guidelines (labels, merge commits)
  • Highlight critical changes (e.g, updated dependencies, credentials)—we want to make sure reviewers are aware of these changes and won't miss them
  • Run non-critical, warning-only checks.

We use hooks heavily in development (and even built our own hook manager—Lefthook), mostly for running linters and tests, i.e. something that is required to pass. We do not use Danger for that (although it's possible), we use it as a code review little helper, not guard)

code of conduct - report abuse