Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: upgrade deps to latest versions #82

Merged
merged 6 commits into from May 10, 2023
Merged

feat: upgrade deps to latest versions #82

merged 6 commits into from May 10, 2023

Conversation

fengmk2
Copy link
Member

@fengmk2 fengmk2 commented May 10, 2023

BREAKING CHANGE: drop Node.js < 14 support

BREAKING CHANGE: drop Node.js < 14 support
@codecov
Copy link

codecov bot commented May 10, 2023

Codecov Report

Patch coverage: 100.00% and project coverage change: +1.20 🎉

Comparison is base (6c5825c) 96.01% compared to head (d5ec621) 97.22%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #82      +/-   ##
==========================================
+ Coverage   96.01%   97.22%   +1.20%     
==========================================
  Files          32       32              
  Lines         603     1331     +728     
  Branches      150      336     +186     
==========================================
+ Hits          579     1294     +715     
- Misses         24       37      +13     
Impacted Files Coverage Δ
app.js 100.00% <100.00%> (ø)
app/extend/context.js 99.12% <100.00%> (+2.22%) ⬆️
app/middleware/securities.js 94.91% <100.00%> (+0.62%) ⬆️
lib/middlewares/csrf.js 100.00% <100.00%> (ø)
lib/safe_redirect.js 98.68% <100.00%> (+2.68%) ⬆️
lib/utils.js 87.57% <100.00%> (+1.20%) ⬆️

... and 26 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

if (contains(ip)) {
return false;
for (const ipAddress of ipAddresses) {
// FIXME: should support ipv6
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#83

@fengmk2 fengmk2 merged commit c3ca817 into master May 10, 2023
15 checks passed
@fengmk2 fengmk2 deleted the drop-node-less-14 branch May 10, 2023 10:39
fengmk2 pushed a commit that referenced this pull request May 10, 2023
[skip ci]

## [3.0.0](v2.11.0...v3.0.0) (2023-05-10)

### ⚠ BREAKING CHANGES

* drop Node.js < 14 support

### Features

* upgrade deps to latest versions ([#82](#82)) ([c3ca817](c3ca817))
@github-actions
Copy link

🎉 This PR is included in version 3.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

fengmk2 added a commit to eggjs/egg that referenced this pull request May 10, 2023
fengmk2 added a commit to eggjs/egg that referenced this pull request May 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant