loading...

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

TOP OF THREAD FULL DISCUSSION
re: 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...
 

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