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

Update dependencies and travis matrix #446

Merged
merged 6 commits into from Jun 4, 2018

Conversation

rymai
Copy link
Member

@rymai rymai commented Jun 4, 2018

  • 38108c5 2018-06-03 | Replace jruby-9.1.2.0 by jruby-9.2.0.0 in the build matrix [Olle Jonsson]
  • 913c166 2018-06-04 | Simplify the build matrix to use minor Ruby versions [Rémy Coutable]
  • 736dd9c 2017-12-31 | Add Ruby 2.5.0 to the build matrix [Tsukuru Tanimichi]
  • 1161552 2016-06-05 | Replace rbx-2 with rbx-3 in the build matrix [Omer Katz]
  • 45802c5 2018-06-04 | Don't use RubyDep anymore since it's outdated and not very useful [Rémy Coutable]

Copy link

@houndci-bot houndci-bot left a comment

Choose a reason for hiding this comment

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

Some files could not be reviewed due to errors:

.rubocop.yml: Style/DotPosition has the wrong namespace - should be Layout
.rubocop.yml: Style/DotPosition has the wrong namespace - should be Layout
.rubocop.yml: Style/FileName has the wrong namespace - should be Naming
.rubocop.yml: Style/PredicateName has the wrong namespace - should be Naming
.rubocop.yml: Style/AccessorMethodName has the wrong namespace - should be Naming
Error: The `Style/TrailingCommaInLiteral` cop no longer exists. Please use `Style/TrailingCommaInArrayLiteral` and/or `Style/TrailingCommaInHashLiteral` instead.
(obsolete configuration found in .rubocop.yml, please update it)
obsolete parameter MaxLineLength (for Style/IfUnlessModifier) found in .rubocop.yml
`Style/IfUnlessModifier: MaxLineLength` has been removed. Use `Metrics/LineLength: Max` instead

rymai and others added 6 commits June 4, 2018 10:54
Signed-off-by: Rémy Coutable <remy@rymai.me>
Signed-off-by: Rémy Coutable <remy@rymai.me>
Signed-off-by: Rémy Coutable <remy@rymai.me>
Signed-off-by: Rémy Coutable <remy@rymai.me>
Signed-off-by: Rémy Coutable <remy@rymai.me>
Signed-off-by: Rémy Coutable <remy@rymai.me>
@rymai rymai force-pushed the update-dependencies-and-travis-matrix branch from 38108c5 to 648c64f Compare June 4, 2018 08:54
@coveralls
Copy link

coveralls commented Jun 4, 2018

Coverage Status

Coverage remained the same at 97.676% when pulling 648c64f on update-dependencies-and-travis-matrix into daac99d on master.

@junaruga
Copy link
Contributor

junaruga commented Jun 4, 2018

@rymai thanks for the PR!

https://travis-ci.org/guard/listen/builds/387684506?utm_source=github_status&utm_medium=notification

Ruby 2.4 -> error by simplecov. Random error?
JRuby: failed tests as we already know.

But it looks good to me. I think this PR improves current master branch.

/home/travis/build/guard/listen/vendor/bundle/ruby/2.4.0/gems/simplecov-0.14.1/lib/simplecov/source_file.rb:87:in `close': Bad file descriptor @ fptr_finalize_flush - /home/travis/build/guard/listen/spec/lib/listen/silencer/controller_spec.rb (Errno::EBADF)

https://rubygems.org/gems/simplecov
simplecov latest version is 1.16.1. But simplecov 0.14.1 is used by latest coveralls.

@olleolleolle
Copy link
Contributor

Someone cared to PR the coveralls gem:
lemurheavy/coveralls-ruby#129

@rymai rymai merged commit 9f14073 into master Jun 4, 2018
@rymai rymai deleted the update-dependencies-and-travis-matrix branch June 4, 2018 12:53
tnir added a commit to tnir/rails_sample_app that referenced this pull request May 19, 2021
See guard/listen#446 and
guard/listen#490

Signed-off-by: Takuya Noguchi <takninnovationresearch@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants