Closed thalesmg closed 3 months ago
Hi @dkataskin ! Have you had the opportunity to consider this? :smiley_cat:
@dkataskin would you be willing to accept PRs that remove this usage?
@thalesmg Yes, I think it makes sense.
Awesome! I'll try to prepare a couple PR in the coming weeks. Thanks! :beers:
Hi! Thanks for your work on this library! :beers:
I'm evaluating this library for usage in our project, and I noticed that apparently there's no strict need to use a
gen_server
for the API calls to Azure. From what I could understand, it appears to me that the only goal of using agen_server
process is to hold the credentials in its state. Please correct me if I'm mistaken. :sweat_smile:Would you be willing to accept future PRs that remove the use of
gen_server
and instead just uses a context map or record to hold the necessary info?For example:
With this, there's no more need to serialize all calls through a single process, nor the need to add it to any supervision tree.
And since we are on the acceptable PR topic, would you be willing to accept PRs that format the source code using
erlfmt
?