Skip to content

[Port dspace-9_x] Use several modern Node.js patterns#4711

Merged
tdonohue merged 3 commits intodspace-9_xfrom
backport-4632-to-dspace-9_x
Sep 10, 2025
Merged

[Port dspace-9_x] Use several modern Node.js patterns#4711
tdonohue merged 3 commits intodspace-9_xfrom
backport-4632-to-dspace-9_x

Conversation

@dspace-bot
Copy link
Contributor

Port of #4632 by @alanorth to dspace-9_x.

Use the the `node:` protocol when importing Node.js builtin modules
and enable rule in eslint to enforce.

The `node:` protocol for Node.js builtin modules has been available
since Node.js v14.

See: https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/enforce-node-protocol-usage.md
(cherry picked from commit af71c15)
We can use the Node.js Fetch API directly instead of relying on an
external dependency.

(cherry picked from commit 2d31afb)
Modern Node.js can watch natively so we don't need nodemon anymore.

(cherry picked from commit 4d70510)
@dspace-bot dspace-bot added 1 APPROVAL pull request only requires a single approval to merge code task dependencies Pull requests that update a dependency file improvement labels Sep 10, 2025
@tdonohue tdonohue added this to the 9.2 milestone Sep 10, 2025
@tdonohue tdonohue merged commit 8ecde49 into dspace-9_x Sep 10, 2025
15 checks passed
@tdonohue tdonohue deleted the backport-4632-to-dspace-9_x branch September 10, 2025 18:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1 APPROVAL pull request only requires a single approval to merge code task dependencies Pull requests that update a dependency file improvement

Projects

Development

Successfully merging this pull request may close these issues.

3 participants