Skip to content

Commit

Permalink
Merge pull request #16 from DannyBen/drop/ruby2-support
Browse files Browse the repository at this point in the history
Drop support for Ruby 2.x
  • Loading branch information
DannyBen committed Jun 30, 2023
2 parents aa6d7ac + 241e8db commit bac545e
Show file tree
Hide file tree
Showing 4 changed files with 9 additions and 5 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ jobs:
runs-on: ubuntu-latest

strategy:
matrix: { ruby: ['2.6', '2.7', '3.0', '3.1'] }
matrix: { ruby: ['3.0', '3.1', '3.2', 'head'] }

steps:
- name: Checkout code
Expand Down
6 changes: 5 additions & 1 deletion .rubocop.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,12 @@ inherit_gem:
- rspec.yml

AllCops:
TargetRubyVersion: 2.6
TargetRubyVersion: 3.0

# Defining `respond_to_missing?` with `true` breaks specs
Style/MissingRespondToMissing:
Enabled: false

# Allow using `file1` in spec let
RSpec/IndexedLet:
Enabled: false
4 changes: 2 additions & 2 deletions lib/sting/sting_operations.rb
Original file line number Diff line number Diff line change
Expand Up @@ -11,10 +11,10 @@ def <<(source)
content = source.transform_keys(&:to_s)

elsif source.include? '*'
Dir[source.to_s].sort.each { |file| push file }
Dir[source.to_s].each { |file| push file }

elsif File.directory? source
Dir["#{source}/*.yml"].sort.each { |file| push file }
Dir["#{source}/*.yml"].each { |file| push file }

else
source = "#{source}.yml" unless /\.ya?ml$/.match?(source)
Expand Down
2 changes: 1 addition & 1 deletion sting.gemspec
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ Gem::Specification.new do |s|
s.files = Dir['README.md', 'lib/**/*.*']
s.homepage = 'https://github.com/dannyben/sting'
s.license = 'MIT'
s.required_ruby_version = '>= 2.6.0'
s.required_ruby_version = '>= 3.0'

s.add_runtime_dependency 'extended_yaml', '~> 0.2'
s.metadata['rubygems_mfa_required'] = 'true'
Expand Down

0 comments on commit bac545e

Please sign in to comment.