Skip to content

fix: add missing options to Client.new/1 and start_link/2 specs - #32

Open
klacointe wants to merge 1 commit into
nutshell-lab:mainfrom
klacointe:fix/client-new-spec
Open

klacointe wants to merge 1 commit into
nutshell-lab:mainfrom
klacointe:fix/client-new-spec

Conversation

@klacointe

Copy link
Copy Markdown

The specs declared four options while the implementations read seven. :debug, :finch and :adapter were missing; :finch and :adapter are both documented in the README, and :finch is needed for the default Tesla.Adapter.Finch to work at all. Because no realistic call matched the declared spec, Dialyzer reported no_return on every caller that builds a client.

Typespec-only change: no runtime behaviour is affected.

The specs declared four options while the implementations read seven.
:debug, :finch and :adapter were missing; :finch and :adapter are both
documented in the README, and :finch is needed for the default
Tesla.Adapter.Finch to work at all. Because no realistic call matched
the declared spec, Dialyzer reported no_return on every caller that
builds a client.

Typespec-only change: no runtime behaviour is affected.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant