New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Dev] Adds Danger #184

Merged
merged 1 commit into from Jun 23, 2017

Conversation

Projects
None yet
1 participant
@orta
Member

orta commented Jun 23, 2017

This is going to be working off our staging peril instance, instead of running on CI. So it might take a bit of setup.

You can see the peril settings here: https://github.com/artsy/artsy-danger/blob/master/peril.settings.json

If you want to understand how all this works, see: https://github.com/danger/peril/blob/master/docs/setup_for_org.md

@artsy-peril

This comment has been minimized.

Show comment
Hide comment
@artsy-peril

artsy-peril bot Jun 23, 2017

Fails
🚫

Please assign someone to merge this PR, and optionally include people who should review.

Generated by 🚫 dangerJS

artsy-peril bot commented Jun 23, 2017

Fails
🚫

Please assign someone to merge this PR, and optionally include people who should review.

Generated by 🚫 dangerJS

@orta

This comment has been minimized.

Show comment
Hide comment
@orta

orta Jun 23, 2017

Member

Going to self-merge this, and retain the fail from Peril on purpose - for archival reasons 👍

Member

orta commented Jun 23, 2017

Going to self-merge this, and retain the fail from Peril on purpose - for archival reasons 👍

@orta orta merged commit 09fbec0 into master Jun 23, 2017

1 check passed

ci/circleci Your tests passed on CircleCI!
Details

@orta orta deleted the danger branch Jun 23, 2017

@orta orta referenced this pull request Apr 22, 2018

Open

Peril Highlights #235

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment