Skip to content
Merged
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions couchbase-orm.gemspec
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@ Gem::Specification.new do |gem|
gem.add_runtime_dependency 'couchbase', '>= 3.4.2'
gem.add_runtime_dependency 'radix', '~> 2.2' # converting numbers to and from any base
gem.add_runtime_dependency 'json-schema', '>= 3' # validating JSON against a schema
gem.add_runtime_dependency 'logger' # Required from Ruby 3.5.0+

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

While adding logger as a dependency is correct, it's a good practice to specify a version constraint to ensure build reproducibility and prevent accidentally pulling in future versions with breaking changes. I'd suggest adding a pessimistic version constraint and aligning it with the other dependencies for consistency.

  gem.add_runtime_dependency     'logger',       '~> 1.6.0' # Required from Ruby 3.5.0+


gem.add_development_dependency 'rake', '~> 12.2'
gem.add_development_dependency 'rspec', '~> 3.7'
Expand Down