Skip to content

Commit

Permalink
Merge pull request #17 from mrship/bump-dry-core
Browse files Browse the repository at this point in the history
Bump dry core and cater for running under Ruby 3
  • Loading branch information
flash-gordon authored Jan 16, 2025
2 parents 762541f + c1ea4c8 commit dd72dee
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion lib/rom/yesql/gateway.rb
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,7 @@ class Gateway < ROM::Gateway
# @option :query_proc [Proc]
#
# @api public
def initialize(*)
def initialize(*, **)
super
@connection = Sequel.connect(uri, options)
@queries = @queries.merge(load_queries(path)).freeze
Expand Down
2 changes: 1 addition & 1 deletion rom-yesql.gemspec
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ Gem::Specification.new do |spec|
spec.require_paths = ["lib"]

spec.add_runtime_dependency "rom", "~>5"
spec.add_runtime_dependency "dry-core", "~>0.4"
spec.add_runtime_dependency "dry-core", "~>1.0"
spec.add_runtime_dependency "sequel", "~>5"
spec.add_development_dependency "bundler"
spec.add_development_dependency "rake", "~> 10.0"
Expand Down
2 changes: 1 addition & 1 deletion spec/integration/adapter_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
let(:container) { ROM.container(configuration) }

let!(:configuration) do
ROM::Configuration.new(:yesql, [uri, path: path, queries: {reports: report_queries}])
ROM::Configuration.new(:yesql, *[uri, path: path, queries: {reports: report_queries}])

Check failure on line 11 in spec/integration/adapter_spec.rb

View workflow job for this annotation

GitHub Actions / Lint

Lint/RedundantSplatExpansion: Pass array contents as separate arguments.

Check failure on line 11 in spec/integration/adapter_spec.rb

View workflow job for this annotation

GitHub Actions / Style

Style/HashAsLastArrayItem: Wrap hash in `{` and `}`.
end

let(:report_queries) { {all_users: "SELECT * FROM users ORDER BY %{order}"} }
Expand Down

0 comments on commit dd72dee

Please sign in to comment.