Skip to content
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

Add Danger as a collaboration tool #4

Merged
merged 4 commits into from
Jan 28, 2017
Merged

Add Danger as a collaboration tool #4

merged 4 commits into from
Jan 28, 2017

Conversation

michaelherold
Copy link
Owner

Danger helps open source maintainers to ensure that certain things are
done by contributors. This helps the maintainers more easily check to
make sure that contributors (including themselves) are following the
contributing guidelines.

@michaelherold michaelherold changed the title Adds Danger as a collaboration tool Add Danger as a collaboration tool Jan 28, 2017
Danger helps open source maintainers to ensure that certain things are
done by contributors. This helps the maintainers more easily check to
make sure that contributors (including themselves) are following the
contributing guidelines.
By enabled Danger, we enable the installation of Rubocop on the CI
environment. Rubocop depends on the "rainbow" library which has a build
error on certain versions of RubyGems. This has been fixed in newer
versions of RubyGems, so let's just update RubyGems prior to trying to
install anything.

See: ku1ik/rainbow#44
See: rubygems/rubygems#1819
This is related to the RakeBuilder problem with building the rainbow
library. Basically, Rake needs to be installed in order to build
external dependencies.

This is an attempt to make sure that Rake is installed.
@michaelherold michaelherold force-pushed the add-danger branch 2 times, most recently from 7828780 to 7e1ce2f Compare January 28, 2017 19:41
jruby-9.0.0.0 was having build errors with pry, so let's try a newer
version.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant