annotate_models icon indicating copy to clipboard operation
annotate_models copied to clipboard

codeclimate-test-reporter (deprecated) shows warnings

Open nard-tech opened this issue 6 years ago • 3 comments

codeclimate-test-reporter (deprecated) shows a warning message.

In addition, other warnings are displayed due to an old version of simplecov (0.12.0), on which codeclimate-test-reporter depends.

I tried to remove these messages, but it seems to be impossible because configuring CircleCI for annotate_models is required.

Commands

$ bundle exec rspec

After tests finished, the following messages is displayed.

Finished in 0.38752 seconds (files took 1.18 seconds to load)
135 examples, 0 failures, 9 pending

.../annotate_models/.gems/gems/simplecov-0.12.0/lib/simplecov/configuration.rb:207: warning: constant ::Fixnum is deprecated
[Coveralls] Outside the CI environment, not sending data.
.../annotate_models/.gems/gems/simplecov-0.12.0/lib/simplecov/source_file.rb:29: warning: constant ::Fixnum is deprecated
.../annotate_models/.gems/gems/simplecov-0.12.0/lib/simplecov/source_file.rb:30: warning: constant ::Fixnum is deprecated
Coverage report generated for RSpec to .../annotate_models/coverage. 8239 / 22117 LOC (37.25%) covered.
Formatter CodeClimate::TestReporter::Formatter failed with NoMethodError: undefined method `map' for #<SimpleCov::Result:0x007fb03ced2530>
Did you mean?  tap (.../annotate_models/.gems/gems/codeclimate-test-reporter-1.0.9/lib/code_climate/test_reporter/formatter.rb:16:in `format')
.../annotate_models/.gems/gems/simplecov-0.12.0/lib/simplecov/configuration.rb:207: warning: constant ::Fixnum is deprecated

Version

  • annotate version: 2.7.3
  • rails version: 5.2.2
  • ruby version: 2.6.0

nard-tech avatar Feb 02 '19 08:02 nard-tech

I didn't see any mention of CircleCI in the output. But is the fix here to migrate to the non-ruby version of test-reporter??

drwl avatar Apr 30 '19 04:04 drwl

I'm able to reproduce locally using ruby 2.6.1. I don't think this project uses CircleCI but @ctran can confirm?

drwl avatar Sep 30 '19 21:09 drwl

We don't need CircleCI. Where is the PR that shows this?

ctran avatar Oct 03 '19 20:10 ctran