使用 Danger 提高 Code Review 体验

Danger是一款用于自动化代码审查的工具，能够简化代码审查流程并确保代码质量。通过简单的配置即可实现如检查PR目标分支、commit消息格式等自动化审查任务。

摘要生成于 C知道，由 DeepSeek-R1 满血版支持，前往体验 >

本文将介绍 Danger 是什么以及如何使用、配置。Demo 地址 DangerDemo。

在 Code Review 时，我们可能经常要去检查各种事情，比如 pr 是否提到了 develop 分支、commit 中是否有毒（存在 merge commit）、禁止某些文件在 pr 中有修改、pr 的描述是否正常等等各种事情。有时我们会忘记检查这些事情，merge 之后才发现，这个就非常尴尬了。

Danger 可以友好地完成上述需求，而且 Danger 配置非常简单，很容易集成到现有 CI （如 Jenkins 、Travis）中。

比如，pr 标题添加[WIP]表示这个 pr 未完成，添加如下代码即可在 pr 中提示我们，这是一个未完成的 pr 。

1	warn( "PR is classed as Work in Progress") if github.pr_title. include? "[WIP]"

集成 Danger 到你的项目

集成 Danger 的指导在 http://danger.systems/guides/getting_started.html 。

一共 6 步。

集成 Danger 到工程中

创建 Gemfiles ：

1	bundle init

在 Gemfiles 中添加 Danger ，示例如下：

     
      source 'https://rubygems.org'
     
     
     
      gem '
      danger'

执行：

1	bundle install

接下来你可以直接执行bundle exec danger init根据指导进行一步一步操作，当然你也可以继续阅读后面的集成步骤。

=。= 你也可以执行后一路回车继续看下面的内容。这一步执行除了添加 Dangerfile ，剩下的都是每一步的指导。

创建 Dangerfile

在根目录创建一个 Dangerfile ，示例文件如下：

     
      # Sometimes it's a README fix, or something like that - which isn't relevant for
     
     
      # including in a project's CHANGELOG for example
     
     
      declared_trivial = github.pr_title.
      include? 
      "#trivial"
     
     
     
      # Make it more obvious that a PR is a work in progress and shouldn't be merged yet
     
     
      warn(
      "PR is classed as Work in Progress") 
      if github.pr_title.
      include? 
      "[WIP]"
     
     
     
      # Warn when there is a big PR
     
     
      warn(
      "Big PR") 
      if git.lines_of_
      code > 
      500
     
     
     
      # Don't let testing shortcuts get into master by accident
     
     
      fail(
      "fdescribe left in tests") 
      if 
      `grep -r fdescribe specs/ `.length > 
      1
     
     
      fail(
      "fit left in tests") 
      if 
      `grep -r fit specs/ `.length > 
      1

事实上 Dangerfile 内容是一段 Ruby 代码。

创建一个 GitHub 账户（可选）

这一步是可选的，你可以为你的项目创建一个机器人账户，你也可以使用自己的账户，更推荐使用一个机器人账户，注册过程和注册一个正常的账号没有区别。

这里我们使用自己的账户。

配置一个 Access Token

创建完 GitHub 账户后，我们需要为该账户添加一个 Access Token 提供给 Danger 使用（读写 GitHub Repo，评论 PR 等功能）。

使用该链接创建对应一个新的 Token https://github.com/settings/tokens/new。

如果你的项目是开源项目，你只需要提供 public_repo 权限，如果你的项目是个私有项目，你可能需要提供完整的 repo 权限（即 repo:status repo_deployment public_repo）。

将 Token 添加到 CI 中

在 Travis CI 中添加环境变量非常容易。添加 DANGER_GITHUB_API_TOKEN 变量即可，对应 Value 是在上一步得到的 Token 。

如果你的项目是一个开源项目，你可能需要打开 Display value in build log 选项确保一个 fork 项目的 PR 可以正常的工作。

对于其他 CI 你可能需要在 GETTING SET UP 中了解更多。

你还需要在你的 Travis 配置中添加 Danger 的运行：

     
      osx_image: xcode8.
      2
     
     
      language: objective-c
     
     
      before_install:
     
     
        - 
      bundle 
      install
     
     
      script:
     
     
        - xcodebuild 
      build -project DangerDemo.xcodeproj -
      scheme DangerDemo -sdk iphonesimulator10.
      2
     
     
        - 
      bundle exec 
      danger

完成以上步骤，你可以创建一个新的 PR 验证 Danger 是否正常运行。

你可以创建一个带 [WIP] 标题的 PR ，此时你应该会在该 PR 中得到如下评论：

配置 Danger

事实上配置的过程就是在 Danger 中加一些你需要的 Ruby 代码。

你可以在http://danger.systems/reference.html中找到完整的 API 。

对 PR 的评论可以添加三种表述，warn fail message ，你可以根据情况去调用，fail 会设置对应的 PR status 为 fail 。

我在这里列出几个比较实用的例子，你可以直接添加进去：

PR 需要提到 develop ：

1	warn( "Please target PRs to `develop` branch") if github.branch_for_base != "develop"

将 warn 改为 fail ，并在 repo 上设置好对应的权限，就可以更好地保证未提到 develop 的代码不小心 merge 了。

一定要为 PR 添加相应的描述：

1	fail "Please provide a summary in the Pull Request description" if github.pr_body.length < 5

某些配置文件被修改是添加 Message ：

     
      if git.modified_files.
      include? 
      "config/*.js"
     
     
        config_files = git.modified_files.select { 
      |path| path.
      include? 
      "config/" }
     
     
        message 
      "This PR changes #{ github.html_link(config_files) }"
     
     
      end

保证 commit 历史是清晰的，没有 merge commit 存在：

     
      if git.commits.any? { 
      |c| c.message =~ 
      /^Merge branch '#{github.branch_for_base}'/ }
     
     
        fail(
      'Please rebase to get rid of the merge commits in this PR')
     
     
      end

Danger 支持插件，你可以自己创建需要的插件，这里还有一些为 iOS 项目提供的插件供添加：

注意 Dangerfile 只是一段 Ruby 代码，你可以添加任何你需要的需求。

这里还有一些已有项目的 Dangerfile ：