Skip to content
This repository has been archived by the owner on Jul 4, 2018. It is now read-only.

Adding coveralls.io #20

Merged
merged 4 commits into from
Jul 12, 2017
Merged

Adding coveralls.io #20

merged 4 commits into from
Jul 12, 2017

Conversation

opsinaka
Copy link

No description provided.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling c8f7a3e on nacho.coveralls into ** on master**.

Gemfile Outdated
gemspec
gem 'coveralls', require: false
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

shouldn't it be required only in test mode?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

perhaps we could add this to the gemspec

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 698cd09 on nacho.coveralls into ** on master**.

@flaviogranero flaviogranero merged commit e0e054a into master Jul 12, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants