We want to be careful to avoid accidentally bringing
in runtime dependencies to projects which use Docile.
In this PR, we clarify explicitly that the CI-only dependency
on simplecov-cobertura, which is the formatt used by CodeCov,
is a test dependency, and not a runtime dependency.
Without this change, it may be possible that projects which use
Docile, when running in an environment with CI=true, would
accidentally bring in this test-only dependency, when that isn't
intended.
- Remove unnecessary quoting of '3.1' in .github/workflows/main.yml
This should finally allow us to adopt code style lints, as well
as others such as performance linting, and have them enforced by
the Github Actions continuous integration (CI) jobs.
For now, I'm choosing to depend on the 'panolint' gem from
Panorama Education, which is my current workplace, as I'd like
to adopt and stay consistent with the setting used there.
Changes
* Move development and test dependencies from gemspec to Gemfile
* Add dependency on 'panolint' gem
* Add .rubocop.yml
* Fix all existing issues
* Run rubocop in github actions CI
Re: #60
Good bye to Travis CI! You were very appreciated.
But today, supporting only Github Actions for CI will enable
some additional goals, as well as speeding up builds.
Based on: https://github.com/ruby/setup-ruby
Re: #60
Some notes on the initial Github Actions config:
1. Set min ruby version to 2.5 in order to support JRuby,
which as of 9.2.17.0 has a RUBY_VERSION of 2.5.
2. For CodeCov exclude JRuby and TruffleRuby, to avoid
errors sending the results in Github Actions on those
jobs rather than debugging them further.
3. Plan is to next remove Travis CI, and then to convert
the CodeCov config to use the Github Action for CodeCov
(thanks @taichi-ishitani for this suggestion!)
In this PR:
* Temporarily remove all YARD doc config, to be added back
after migrating CI to Github Actions, to simplify config
and dependencies
* Remove all testing and references to < 2.6 Rubies
* Simplify JRuby and TruffleRuby testing to latest stable versions
NOT in this PR:
* Any simplifying code changes that are now possible due to
removing support for EOL Rubies
Closes#58
