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

Remove unnecessary dependencies #10

Open
wants to merge 9 commits into
base: develop
Choose a base branch
from

Conversation

dwilkie
Copy link

@dwilkie dwilkie commented Aug 4, 2016

This PR removes i18n and adhearsion as runtime dependencies. This helped me when using this plugin with the latest version of Adherasion

@benlangfeld
Copy link
Member

The dependency on adhearsion is not unnecessary; see adhearsion/voicemail#46 for the preferred solution.

The dependency on adhearsion-i18n can be removed, however.

dwilkie added 5 commits August 5, 2016 13:09
This conversion is done by Transpec 3.2.2 with the following command:
    transpec

* 25 conversions
    from: obj.should
      to: expect(obj).to

* 21 conversions
    from: == expected
      to: eq(expected)

* 2 conversions
    from: lambda { }.should
      to: expect { }.to

* 1 conversion
    from: collection.should have(n).items
      to: expect(collection.size).to eq(n)

* 1 conversion
    from: obj.should_not_receive(:message)
      to: expect(obj).not_to receive(:message)

* 1 conversion
    from: obj.should_receive(:message)
      to: expect(obj).to receive(:message)

For more details: https://github.com/yujinakayama/transpec#supported-conversions
@dwilkie
Copy link
Author

dwilkie commented Aug 5, 2016

@benlangfeld thanks. I re-added the dependency on Adhearsion and set it to 3.0.0.rc1


s.add_development_dependency "rspec", "~> 2.7.0"
s.add_development_dependency "rspec"
Copy link
Member

Choose a reason for hiding this comment

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

Why remove these version specs?

Copy link
Author

Choose a reason for hiding this comment

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

I get the following error if I keep thor at "~> 0.14.0"

adhearsion-drb was resolved to 1.1.1, which depends on
adhearsion (> 3.0.0.rc1) was resolved to 3.0.0.rc1, which depends on
thor (
> 0.18.0)

guard-rspec was resolved to 4.7.3, which depends on
  guard (~> 2.1) was resolved to 2.14.0, which depends on
    thor (>= 0.18.1)

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.

2 participants