Skip to content

docs(readme): zlib deprecation prevention - #119

Merged
niftylettuce merged 1 commit into
koajs:masterfrom
Leafgard:patch-1
Jul 6, 2020
Merged

niftylettuce merged 1 commit into
koajs:masterfrom
Leafgard:patch-1

Conversation

@Leafgard

@Leafgard Leafgard commented Jul 4, 2020 •

Copy link
Copy Markdown

Access Z_SYNC_FLUSH is still possible but deprecated as you can see on Zlib documentation.

Previously, the constants were available directly from require('zlib'), for instance zlib.Z_NO_FLUSH. Accessing the constants directly from the module is currently still possible but is deprecated.

Prevented this deprecation by accessing Z_SYNC_FLUSH from constants namespace.

Access Z_SYNC_FLUSH is still possible but deprecated. Prevented this deprecation by accessing Z_SYNC_FLUSH from constants namespace.
@codecov

codecov Bot commented Jul 4, 2020

Copy link
Copy Markdown

Codecov Report

Merging #119 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##            master      #119   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            2         2           
  Lines           80        80           
  Branches        29        29           
=========================================
  Hits            80        80           

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 cce99c4...e341e0f. Read the comment docs.

@niftylettuce
niftylettuce merged commit 22c434c into koajs:master Jul 6, 2020
@niftylettuce

Copy link
Copy Markdown
Contributor

v5.0.1 released to npm and GitHub

https://github.com/koajs/compress/releases/tag/v5.0.1

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.

2 participants