Closed fengmk2 closed 3 months ago
[!WARNING]
Rate limit exceeded
@fengmk2 has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 8 minutes and 16 seconds before requesting another review.
How to resolve this issue?
After the wait time has elapsed, a review can be triggered using the `@coderabbitai review` command as a PR comment. Alternatively, push new commits to this PR. We recommend that you space out your commits to avoid hitting the rate limit.How do rate limits work?
CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our [FAQ](https://coderabbit.ai/docs/faq) for further information.Commits
Files that changed from the base of the PR and between bfd7819fdcb355966d8fa704e635629946033f4e and 04c24de6f7f834eeff671489b3fccaab05b2ad64.
The updates involve modernizing dependencies, refactoring modules, and enhancing configurations to support newer Node.js versions. Major changes include updating to Node.js versions 18, 20, and 22, refining import syntax, removing outdated patterns, and improving error handling. The documentation and configuration policies were also aligned to reflect these enhancements, ensuring higher compatibility and streamlined processes.
Files | Change Summary |
---|---|
.eslintrc | Removed parserOptions and ignorePatterns configurations |
.github/workflows/nodejs.yml | Updated Node.js versions to 18.19.0, 20, 22 |
README.md | Replaced contributor list with dynamic badge and updated license section |
package.json | Updated version, dependencies, node engine requirement, scripts, and exports section |
scripts/replace_urllib_version.js | Updated replacement logic and imports |
src/HttpClient.ts | Refactored imports, removed unused functions, and updated HEADER_USER_AGENT |
src/diagnosticsChannel.ts | Simplified functions, updated type handling, and enhanced error handling |
src/index.ts | Changed import of LRU to a named import |
src/utils.ts | Explicitly typed JSONCtlCharsMap and removed sleep function from exports |
test/diagnostics_channel.test.ts | Updated import paths and added setTimeout from node:timers/promises |
test/fixtures/server.ts | Added import for setTimeout as sleep |
test/keep-alive-header.test.ts | Updated import statements and renamed sleep function |
test/options.signal.test.ts | Adjusted imports to use explicit .js extensions and added setTimeout as sleep |
test/options.timeout.test.ts | Simplified error handling logic |
test/options.timing.test.ts | Added setTimeout import and adjusted other imports |
test/options.writeStream.test.ts | Updated imports with .js extensions and added setTimeout as sleep |
test/utils.ts | Removed the sleep function and modified readableToBytes |
tsconfig.json | Changed noImplicitAny to true , removed several options, and retained module and moduleResolution settings |
In codeโs bright light, the changes gleam,
A clearer path, a modern dream,
With Nodeโs new strength and modules refined,
Compatibility we surely find.
Dependencies fresh, no patterns old,
Our projectโs future now unfolds.
๐
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
New and removed dependencies detected. Learn more about Socket for GitHub โ๏ธ
Package | New capabilities | Transitives | Size | Publisher |
---|---|---|---|---|
npm/eslint-config-egg@13.1.0 | Transitive: environment, eval, filesystem, shell, unsafe | +244 |
36.4 MB | fengmk2 |
npm/undici@6.19.2 | environment, network | 0 |
1.13 MB | matteo.collina |
๐ฎ Removed packages: npm/@types/default-user-agent@1.0.2, npm/@types/pump@1.1.3, npm/default-user-agent@1.0.0, npm/digest-header@1.1.0, npm/eslint-config-egg@12.3.1, npm/form-data-encoder@1.9.0, npm/formdata-node@4.4.1, npm/git-contributor@2.1.5, npm/pump@3.0.0, npm/undici@5.28.4
Attention: Patch coverage is 96.07843%
with 2 lines
in your changes missing coverage. Please review.
Project coverage is 97.85%. Comparing base (
d565da2
) to head (04c24de
). Report is 1 commits behind head on master.
Files | Patch % | Lines |
---|---|---|
src/diagnosticsChannel.ts | 93.54% | 2 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
BREAKING CHANGE: drop Node.js < 18.19.0 support
part of https://github.com/eggjs/egg/issues/3644
Summary by CodeRabbit
New Features
README.md
.HttpClient
.Bug Fixes
Socket
methods.Refactor
exports
andscripts
inpackage.json
for better consistency.subscribe
function in diagnostics channel.Chores
.eslintrc
.tsconfig.json
.Documentation
contributors-img
inREADME.md
.Tests