search

hoangvvo / next-connect

The TypeScript-ready, minimal router and middleware layer for Next.js, Micro, Vercel, or Node.js http/http2
https://www.npmjs.com/package/next-connect
MIT License
1.63k stars 65 forks source link

Deprecate apply() for run() #108

Closed hoangvvo closed 3 years ago

hoangvvo commented 3 years ago

apply() is replaced with run().

It will be removed completely in v1.0.0.

Close #82

changeset-bot[bot] commented 3 years ago

⚠️ No Changeset found

Latest commit: 89cea7de7dc28f8a3ea0ad0936decfcbd7335ac5

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

[Click here if you're a maintainer who wants to add a changeset to this PR](https://github.com/hoangvvo/next-connect/new/refactor/apply-to-run?filename=.changeset/tender-laws-argue.md&value=---%0A%22%40fake-scope%2Ffake-pkg%22%3A%20patch%0A---%0A%0ADeprecate%20apply()%20for%20run()%0A)

codecov[bot] commented 3 years ago

Codecov Report

Merging #108 into master will not change coverage. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##            master      #108   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            1         1           
  Lines           45        47    +2     
=========================================
+ Hits            45        47    +2
Impacted Files Coverage Δ
lib/index.js 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update e4efc6b...89cea7d. Read the comment docs.