search

chulkilee / ex_force

A Salesforce REST API wrapper for Elixir
https://hex.pm/packages/ex_force
MIT License
38 stars 28 forks source link

Fix default headers #46

Closed chulkilee closed 3 years ago

chulkilee commented 3 years ago

The existing code uses the default argument for default value for headers option. This is not working when any option is given (e.g. api_version option is given for example) - then the default value won't be used.

This is a changing behavior, but it shouldn't break anything. I'm not sure it should be minor or patch bump.

coveralls commented 3 years ago

Coverage Status

Coverage remained the same at 100.0% when pulling 9b6ad5241d4bec61f0d6ab9f6e8d81afdb73e8b8 on fix-default-headers into 2a3e81c934111322fa4705e63bb4492f6c243a35 on main.