Skip to content

Comments

refactor(website): remove unused dependencies and update node engine requirement#6421

Merged
Xuanwo merged 1 commit intoapache:mainfrom
kingsword09:refactor-website
Jul 17, 2025
Merged

refactor(website): remove unused dependencies and update node engine requirement#6421
Xuanwo merged 1 commit intoapache:mainfrom
kingsword09:refactor-website

Conversation

@kingsword09
Copy link
Contributor

@kingsword09 kingsword09 commented Jul 17, 2025

Which issue does this PR close?

Closes #.

Rationale for this change

What changes are included in this PR?

This update primarily focuses on removing unused dependencies, upgrading the minimum required Node.js version, and refactoring the image SSR plugin for improved maintainability and performance.

  • Unused dependencies such as ‎axios and ‎clsx have been removed from both ‎package.json and ‎DEPENDENCIES.node.csv.
  • The minimum Node.js version requirement is now set to 20.0.0 because Node.js 18 has reached its end of life EOL as of April 2025 and no longer receives security or maintenance updates. Upgrading to Node.js 20 ensures better security and compatibility with dependencies.
  • The image SSR plugin (‎image-ssr-plugin.js) has been refactored: it now uses modern async APIs (‎fs/promises instead of ‎fs-extra, and ‎fetch instead of ‎axios).

Are there any user-facing changes?

@kingsword09 kingsword09 requested a review from Xuanwo as a code owner July 17, 2025 02:34
@dosubot dosubot bot added size:M This PR changes 30-99 lines, ignoring generated files. releases-note/refactor The PR does a refactor on code or has a title that begins with "refactor" labels Jul 17, 2025
@kingsword09 kingsword09 changed the title refactor(website): remove unused dependencies and update node engine … refactor(website): remove unused dependencies and update node engine requirement Jul 17, 2025
Copy link
Member

@Xuanwo Xuanwo left a comment

Choose a reason for hiding this comment

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

Thank you @kingsword09 for working on this!

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jul 17, 2025
@Xuanwo Xuanwo merged commit 8ed2692 into apache:main Jul 17, 2025
36 checks passed
@kingsword09 kingsword09 deleted the refactor-website branch July 17, 2025 03:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer releases-note/refactor The PR does a refactor on code or has a title that begins with "refactor" size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants