dry-behaviour 0.11.2 → 0.12.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA256:
-  metadata.gz: 232921c25386484f9538bbe49039c133c1de8e2d2cd85b9545a4de3736344385
-  data.tar.gz: 38625102473e08efc17d5162db60e1fc9a90633f81695ecb60f26f395237f976
+SHA1:
+  metadata.gz: eca195e7f40df7dbd0ff8719938bbca1c717c82c
+  data.tar.gz: 820195b2a867f6e5c1a350b991108259b47c5888
 SHA512:
-  metadata.gz: 4c4c5d71d32fcfe4bef6fb13e21ddcd8a32afc63cdbf3bda1dc97428d56332f99e02fd15cb15db9b3ecafb719812e6889587794598dd666335d02e67780b6ae9
-  data.tar.gz: 3d556f70828ded92ad615fd01ab27b1d7b4c70843bf2d16fa10de1a9102da08ea6f3284883aa2c37a2b0c830776c36ea5ed989cb50ef711e29c5dbb2cfa53531
+  metadata.gz: f106a7ad2a2a4548d17aeeb0053cd8875ce8126330b5cd3ae4d51219e73b564a2a6fe29ab9841e31ac3e506e1746013a96dde1d40ed094ecf18fe09e5fa7e4e4
+  data.tar.gz: bb91d87c3442fd30ea5887a33ca604000d1789f9a60907d0a9b3d2f29ab37966a3ff8b9574a22414307c22309d1ea2d2b0e15f4eb8ec606717d2b24ca1dc496c

data/.github/workflows/ci.yml

@@ -9,12 +9,12 @@ jobs:
 
     steps:
     - uses: actions/checkout@v1
-    - name: Set up Ruby 2.6
-      uses: actions/setup-ruby@v1
+    - name: Set up Ruby 2.3.8
+      uses: ruby/setup-ruby@v1
       with:
-        ruby-version: 2.6.x
+        ruby-version: 2.3.8
     - name: Build and test with Rake
       run: |
         gem install bundler
         bundle install --jobs 4 --retry 3
-        bundle exec rake
+        bundle exec rspec

data/.github/workflows/gempush.yml

@@ -15,10 +15,10 @@ jobs:
 
     steps:
     - uses: actions/checkout@master
-    - name: Set up Ruby 2.6
-      uses: actions/setup-ruby@v1
+    - name: Set up Ruby 2.3.8
+      uses: ruby/setup-ruby@v1
       with:
-        version: 2.6.x
+        version: 2.3.8
 
     - name: Publish to GPR
       run: |

data/.tool-versions

@@ -0,0 +1 @@
+ruby 2.3.8

data/README.md

@@ -60,6 +60,45 @@ expect(Protocols::Adder.add(nil, 10)).to eq(10)
 expect(Protocols::Adder.add_default(1)).to eq(6)
 ```
 
+### Arguments types
+
+Normally, one would use a simple notation to declare a method. It includes `:this` receiver
+in te very first position and some optional required arguments afterward.
+
+```ruby
+  defmethod :add, :this, :addend
+  ...
+  defimpl ... do
+    def add(this, addend); this + addend; end
+  end
+```
+
+If the argument is not generic (optional, or splatted, or keyword,) its type must be explicitly
+specified in the protocol definition as shown below.
+
+```ruby
+  defprotocol implicit_inheritance: true do
+    defmethod :with_def_argument, :this, [:foo_opt, :opt]
+    defmethod :with_def_keyword_argument, :this, [:foo_key, :key]
+    defmethod :with_req_keyword_argument, :this, [:foo_key, :keyreq]
+
+    def with_def_argument(this, foo_opt = :super); foo_opt; end
+    def with_def_keyword_argument(this, foo_key: :super); foo_key; end
+    def with_req_keyword_argument(this, foo_key:); foo_key; end
+  ...
+```
+
+That said, `:addend` argument declaration is a syntactic sugar for `[:addend, :req]`.
+Possible values for the type are:
+
+```ruby
+PARAM_TYPES = %i[req opt rest key keyrest keyreq block]
+```
+
+Please note, that the signature of the method and its implementation must exactly match.
+One cannot declare a method to have a `keyreq` argument and then make it defaulted in the
+implementation. That is done by design.
+
 ## Guards
 
 Starting with `v0.5.0` we support multiple function clauses and guards.

data/lib/dry/behaviour/black_tie.rb

@@ -54,15 +54,22 @@ module Dry
         end.reject(&:nil?).first
       end
 
-      BlackTie.protocols[self].each do |method, *_| # FIXME: CHECK ARITY HERE
-        singleton_class.send :define_method, method do |receiver = nil, *args|
+      BlackTie.protocols[self].each do |method, *_m_args, **_m_kwargs| # FIXME: CHECK ARITY HERE
+        singleton_class.send :define_method, method do |receiver, *args, **kwargs|
           impl = implementation_for(receiver)
+
           raise Dry::Protocol::NotImplemented.new(
             :protocol, inspect,
             method: method, receiver: receiver, args: args, self: self
           ) unless impl
+
           begin
-            impl[method].(*args.unshift(receiver))
+            # [AM] [v1] [FIXME] for modern rubies `if` is redundant
+            if kwargs.empty?
+              impl[method].(*args.unshift(receiver))
+            else
+              impl[method].(*args.unshift(receiver), **kwargs)
+            end
           rescue => e
             raise Dry::Protocol::NotImplemented.new(
                 :nested, inspect,
@@ -89,7 +96,7 @@ module Dry
             BlackTie.Logger.warn(UNKNOWN_TYPE_DECLARATION % [Dry::BlackTie.proto_caller, type, self.inspect, name])
             type = nil
           end
-          [type || PARAM_TYPES.include?(p) ? p : :req, p]
+          [type || (PARAM_TYPES.include?(p) ? p : :req), p]
         end
       BlackTie.protocols[self][name] = params
     end
@@ -109,8 +116,13 @@ module Dry
           (NORMALIZE_KEYS.(protocol) - meths).each_with_object(meths) do |m, acc|
             if BlackTie.protocols[protocol][:__implicit_inheritance__]
               mod.singleton_class.class_eval do
-                define_method m do |this, *♿_args, &♿_λ|
-                  super(this, *♿_args, &♿_λ)
+                define_method m do |this, *♿_args, **♿_kwargs, &♿_λ|
+                  # [AM] [v1] [FIXME] for modern rubies `if` is redundant
+                  if ♿_kwargs.empty?
+                    super(this, *♿_args, &♿_λ)
+                  else
+                    super(this, *♿_args, **♿_kwargs, &♿_λ)
+                  end
                 end
               end
             else
@@ -143,7 +155,7 @@ module Dry
     end
     module_function :defimpl
 
-    PARAM_TYPES = %i[req opt rest keyrest block]
+    PARAM_TYPES = %i[req opt rest key keyrest keyreq block]
 
     DELEGATE_METHOD = lambda do |klazz, (source, target)|
       klazz.class_eval do

data/lib/dry/behaviour/version.rb

@@ -1,5 +1,5 @@
 module Dry
   module Behaviour
-    VERSION = '0.11.2'.freeze
+    VERSION = '0.12.0'.freeze
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: dry-behaviour
 version: !ruby/object:Gem::Version
-  version: 0.11.2
+  version: 0.12.0
 platform: ruby
 authors:
 - Aleksei Matiushkin
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-07-08 00:00:00.000000000 Z
+date: 2022-12-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -111,7 +111,7 @@ files:
 - ".rspec"
 - ".rubocop.yml"
 - ".rubocop_todo.yml"
-- ".travis.yml"
+- ".tool-versions"
 - CODE_OF_CONDUCT.md
 - Gemfile
 - LICENSE.txt
@@ -149,7 +149,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.3.3
+rubyforge_project: 
+rubygems_version: 2.5.2.3
 signing_key: 
 specification_version: 4
 summary: Tiny library inspired by Elixir protocol pattern.

data/.travis.yml

@@ -1,7 +0,0 @@
-language: ruby
-rvm:
-  - 2.1.1
-  - 2.1.10
-  - 2.2.2
-  - 2.5.1
-before_install: gem install bundler -v 1.14.6