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
Fix travis ci suite #38
Conversation
….9 (JRuby 1.7) compatible Note: term-ansicolor and tins (sub-dependencies of coveralls) do not follow SemVer
…JRuby 1.7) compatible Note: listen (sub-dependency of guard) does not follow SemVer
NoMethodError: undefined method `last_comment' for #<Rake::Application:0x464e808b> ~/.rvm/gems/jruby-1.7.19/gems/rspec-core-2.99.2/lib/rspec/core/rake_task.rb:143:in `initialize' ~/Code/ruby_speech/Rakefile:7:in `(root)'
…t stable version of Bundler
@@ -1,4 +1,9 @@ | |||
require 'ruby_speech' | |||
# encoding: utf-8 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing frozen string literal comment.
Awesome, thank you Ben!! =) |
Since we can't trust rbx version numbers, I'm happy to make its failure allowed. No-one sane actually uses it anyway. |
Sounds good, marking them as allowed and hoping for a green test suite to come through! =) |
Almost forgot... shall I also include the following key from the adhearsion .travis.yml? notifications:
slack:
secure: Y7EqF/9FY7mca63c1DM7kEr34fNtFclWXJj9vLqVQJwR3rCDZUZSD8DlqHJDwE6KRCCDsSkyV4lihCim+GEM28CRcJir0AEUJhXHvIMckjH7n38W9OGN26RuJGZTapjgCSMOBeq0YpSe6dAnZeRpZRJxsTFtP5ptLD5RRjIMzZM= |
Resolve a number of test-related issues regarding development_dependencies and testing environment setup:
Development_dependency fixes:
Note: term-ansicolor and tins (sub-dependencies of coveralls) broke SemVer by cutting off ruby 1.9 support apart from a major release
Similarly, the listen sub-dependency of guard broke SemVer in this regard as well.
Once we upgrade to rspec 3.0+, we can lift this restriction
Environment setup fixes:
Other changes:
Let me know if this is overkill.
Known and expected issues