Skip to content

remove dev dependency: derequire#9143

Merged
alxndrsn merged 2 commits intoapache:masterfrom
alxndrsn:derequire
Feb 16, 2026
Merged

remove dev dependency: derequire#9143
alxndrsn merged 2 commits intoapache:masterfrom
alxndrsn:derequire

Conversation

@alxndrsn
Copy link
Contributor

Ref https://www.npmjs.com/package/derequire

Not obvious why replacing require(...) with _dereq_(...) is useful.

@alxndrsn alxndrsn marked this pull request as draft December 17, 2025 22:15
@alxndrsn
Copy link
Contributor Author

This might be important - PR changed to draft until the reason for _dereq_ is known.

Maybe:

@SourceR85
Copy link
Contributor

Hi @alxndrsn this looks promising to me.
Do you feel comfortable to mark it ready for review?

@alxndrsn
Copy link
Contributor Author

Hi @alxndrsn this looks promising to me. Do you feel comfortable to mark it ready for review?

I'm still not clear what the intention was for derequire, or if it has any effect, but if you want to review it... #YOLO

@alxndrsn alxndrsn marked this pull request as ready for review February 16, 2026 11:59
Copy link
Contributor

@SourceR85 SourceR85 left a comment

Choose a reason for hiding this comment

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

derequire was used to transpose node's require() to a browser friendly format (_dereq_ by default).

We have replaced all require calls with ES module import in every module, we ship to browsers.

last occurrences can be found in source of pouchdb-adapter-leveldb and pouchdb-for-coverage packages:

  • in LevelDB: requireLeveldown.js, and migrate.js
    (LevelDB is about to be removed: #9163)
  • in Coverage: just for nodejs, because it uses module.exports

@alxndrsn alxndrsn merged commit 5e0c6f2 into apache:master Feb 16, 2026
255 of 259 checks passed
@alxndrsn alxndrsn deleted the derequire branch February 16, 2026 16:04
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

Comments