herbsjs / herbs2rest

Create a REST API based on HerbsJS
MIT License
2 stars 13 forks source link

Feature: async handlers #52

Closed RodolfoSilva closed 1 year ago

RodolfoSilva commented 1 year ago

This will allow the users to implement async handlers when some external communication is needed.

Proposed Changes

  1. Add an await to call the authorizationHandler.
  2. Add an await to call the parametersHandler.

Readiness Checklist

Author/Contributor

Reviewing Maintainer

codecov[bot] commented 1 year ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (beta@f07e0bd). Click here to learn what that means. The diff coverage is n/a.

:exclamation: Current head 140b7e7 differs from pull request most recent head 549826f. Consider uploading reports for the commit 549826f to get more accurate results

@@           Coverage Diff           @@
##             beta      #52   +/-   ##
=======================================
  Coverage        ?   98.91%           
=======================================
  Files           ?        9           
  Lines           ?      276           
  Branches        ?        0           
=======================================
  Hits            ?      273           
  Misses          ?        3           
  Partials        ?        0           

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

herbsjs-robot commented 1 year ago

:tada: This PR is included in version 4.1.0-beta.5 :tada:

The release is available on:

Your semantic-release bot :package::rocket: