Skip to content

Commit 4b5abbc

Browse files
committed
Update GitHub CI
- Remove unused reviewdoc - Remove simplecov-cobertura - simplecov-html is a dependency of simplecov, so we do not need it in gemspec
1 parent 43e837a commit 4b5abbc

File tree

3 files changed

+1
-22
lines changed

3 files changed

+1
-22
lines changed

.github/workflows/ruby.yml

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -31,11 +31,5 @@ jobs:
3131
with:
3232
ruby-version: ${{ matrix.ruby-version }}
3333
bundler-cache: true # runs 'bundle install' and caches installed gems automatically
34-
- name: rubocop
35-
uses: reviewdog/action-rubocop@v2
36-
with:
37-
rubocop_version: gemfile
38-
rubocop_extensions: rubocop-minitest:gemfile rubocop-rake:gemfile rubocop-sequel:gemfile
39-
reporter: github-pr-review
4034
- name: Run tests
4135
run: bundle exec rake

oxidized.gemspec

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -56,8 +56,6 @@ Gem::Specification.new do |s|
5656
s.add_development_dependency 'rubocop-rake', '~> 0.7.0'
5757
s.add_development_dependency 'rubocop-sequel', '~> 0.3.3'
5858
s.add_development_dependency 'simplecov', '~> 0.22.0'
59-
s.add_development_dependency 'simplecov-cobertura', '~> 2.1.0'
60-
s.add_development_dependency 'simplecov-html', '~> 0.13.1'
6159

6260
# Dependencies on optional libraries, used for unit tests & development
6361
s.add_development_dependency 'oxidized-web', '>= 0.15.0'

spec/spec_helper.rb

Lines changed: 1 addition & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -1,18 +1,5 @@
11
require 'simplecov'
2-
require 'simplecov-cobertura'
3-
4-
SimpleCov.start do
5-
if ENV['CI']
6-
formatter SimpleCov::Formatter::CoberturaFormatter
7-
else
8-
formatter SimpleCov::Formatter::MultiFormatter.new(
9-
[
10-
SimpleCov::Formatter::CoberturaFormatter,
11-
SimpleCov::Formatter::HTMLFormatter
12-
]
13-
)
14-
end
15-
end
2+
SimpleCov.start
163

174
require 'minitest/autorun'
185
require 'mocha/minitest'

0 commit comments

Comments
 (0)