Formalize your Pull Request etiquette.
Installation • Usage • DSL • Constraints • Advanced • Contributing
Add this line to your application's Gemfile:
gem 'danger'
To get up and running quickly, just run
bundle exec danger init
bundle exec danger
This will look at your Dangerfile
and update the pull request accordingly. While you are setting up Danger, you may want to use: --verbose
for more debug information.
Danger runs at the end of a CI build, she will execute a Dangerfile
. This file is given some special variables based on the git diff and the Pull Request being running. You can use these variables in Ruby to provide messages, warnings and failures for your build. You set up Danger with a GitHub user account and she will post updates via comments on the Pull Request, and can fail your build too.
Danger 🚫 | ||
---|---|---|
✨ | lines_of_code |
The total amount of lines of code in the diff |
✏️ | modified_files |
The list of modified files |
🚢 | added_files |
The list of added files |
♻️ | deleted_files |
The list of removed files |
🔤 | pr_title |
The title of the PR |
📖 | pr_body |
The body of the PR |
👥 | pr_author |
The author who submitted the PR |
🔖 | pr_labels |
The labels added to the PR |
The Dangerfile
is a ruby file, so really, you can do anything. However, at this stage you might need selling on the idea a bit more, so lets take some real examples:
# Sometimes its a README fix, or something like that - which isn't relevant for
# including in a CHANGELOG for example
declared_trivial = pr_title.include? "#trivial"
# Just to let people know
warn("PR is classed as Work in Progress", sticky: false) if pr_title.include? "[WIP]"
# Devs shouldn't ship changes to this file
fail("Developer Specific file shouldn't be changed", sticky: false) if modified_files.include?("Artsy/View_Controllers/App_Navigation/ARTopMenuViewController+DeveloperExtras.m")
# Did you make analytics changes? Well you should also include a change to our analytics spec
made_analytics_changes = modified_files.include?("/Artsy/App/ARAppDelegate+Analytics.m")
made_analytics_specs_changes = modified_files.include?("/Artsy_Tests/Analytics_Tests/ARAppAnalyticsSpec.m")
if made_analytics_changes
fail("Analytics changes should have reflected specs changes") if !made_analytics_specs_changes
# And pay extra attention anyway
message('Analytics dict changed, double check for ?: `@""` on new entries')
message('Also, double check the [Analytics Eigen schema](https://docs.google.com/spreadsheets/u/1/d/1bLbeOgVFaWzLSjxLOBDNOKs757-zBGoLSM1lIz3OPiI/edit#gid=497747862) if the changes are non-trivial.')
end
message("@orta something changed in elan!") if modified_files.include? "/components/lib/variables/colors.json"
build_log = File.read(File.join(ENV["CIRCLE_ARTIFACTS"], "xcode_test_raw.log"))
snapshots_url = build_log.match(%r{https://eigen-ci.s3.amazonaws.com/\d+/index.html})
fail("There were [snapshot errors](#{snapshots_url})") if snapshots_url
Command | Description |
---|---|
fail |
Causes the PR to fail and print out the error on the PR |
warn |
Prints out a warning to the PR, but still enables the merge button |
message |
Show neutral messages on the PR |
markdown |
Print raw markdown below the summary tables on the PR |
Danger was built with a platform in mind: It can be used with any kind of software project and allows you to write your own action to have structured source code.
In your Dangerfile
you can import local or remote actions using
import "./danger_plugins/work_in_progress_warning"
# or
import "https://raw.githubusercontent.com/danger/danger/master/danger_plugins/work_in_progress_warning.rb"
# Call those actions using
work_in_progress_warning
custom_plugin(variable: "value")
To create a new plugin run
danger new_plugin
This will generate a new Ruby file which you can modify to fit your needs.
Danger currently is supported on Travis CI, Circle CI, BuildKite and Jenkins. These work via environment variables, so it's easy to extend to include your own.
You can access more detailed information by accessing the following variables
Danger 🚫 | |
---|---|
env.request_source.pr_json |
The full JSON for the pull request |
env.scm.diff |
The full Diff file for the diff. |
env.ci_source |
To get information like the repo slug or pull request ID |
These are considered implementation details though, and may be subject to change in future releases. We're very open to turning useful bits into the official API.
Using danger local
will look for the last merged pull request in your git history, and apply your current
Dangerfile
against that Pull Request. Useful when editing.
You can tell Danger to ignore a specific warning or error by commenting on the PR body:
> Danger: Ignore "Developer Specific file shouldn't be changed"
Danger can keep its history if a warning/error/message is marked as sticky. When the violation is resolved,
Danger will update the comment to cross it out. If you don't want this behavior, just use sticky: false
.
fail("PR needs labels", sticky: false) if pr_labels.empty?
- You can set the base branch in the command line arguments see:
bundle exec danger --help
, if you commonly merge into non-master branches. - Appending
--verbose
tobundle exec danger
will expose all of the variables that Danger provides, and their values in the shell.
Here are some real-world Dangerfiles: artsy/eigen, danger/danger, artsy/elan and more!
This project is open source under the MIT license, which means you have full access to the source code and can modify it to fit your own needs.
This project subscribes to the Moya Contributors Guidelines which TLDR: means we give out push access easily and often.
Contributors subscribe to the Contributor Code of Conduct based on the Contributor Covenant version 1.3.0.