[PR #12] [MERGED] fix ESM support - remove require call #16

New issue

Closed

opened 2026-05-23 08:36:43 -06:00 by gitea-mirror · 0 comments

gitea-mirror commented

2026-05-23 08:36:43 -06:00

Owner

📋 Pull Request Information

Original PR: https://github.com/appy-one/acebase-core/pull/12
Author: @akvadrako
Created: 4/28/2022
Status: ✅ Merged
Merged: 4/29/2022
Merged by: @appy-one

Base: master ← Head: fix-esm-no-require

📝 Commits (1)

11d1db2 fix ESM support - remove require call

📊 Changes

5 files changed (+5 additions, -8 deletions)

View changed files

📝 dist/cjs/utils.js (+1 -2)
📝 dist/cjs/utils.js.map (+1 -1)
📝 dist/esm/utils.js (+1 -2)
📝 dist/esm/utils.js.map (+1 -1)
📝 src/utils.ts (+1 -2)

📄 Description

This removes the require() call for ESM support. I couldn't think of a better way to fix this that would also support CJS.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/appy-one/acebase-core/pull/12 **Author:** [@akvadrako](https://github.com/akvadrako) **Created:** 4/28/2022 **Status:** ✅ Merged **Merged:** 4/29/2022 **Merged by:** [@appy-one](https://github.com/appy-one) **Base:** `master` ← **Head:** `fix-esm-no-require` --- ### 📝 Commits (1) - [`11d1db2`](https://github.com/appy-one/acebase-core/commit/11d1db2f2f1bedf3cb5db6cc3d04759e42102799) fix ESM support - remove require call ### 📊 Changes **5 files changed** (+5 additions, -8 deletions) <details> <summary>View changed files</summary> 📝 `dist/cjs/utils.js` (+1 -2) 📝 `dist/cjs/utils.js.map` (+1 -1) 📝 `dist/esm/utils.js` (+1 -2) 📝 `dist/esm/utils.js.map` (+1 -1) 📝 `src/utils.ts` (+1 -2) </details> ### 📄 Description This removes the `require()` call for ESM support. I couldn't think of a better way to fix this that would also support CJS. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>