Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rename datatype.js to dataType.js #8

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Denys-Bushulyak
Copy link

Fix the error that happens on case-sensitive file systems.

Problem founded when testing at node:slim docker image.

@Denys-Bushulyak
Copy link
Author

while PR pending - temporal solution:

install npm patch-package and apply at patch file <your project root>/patches/node-pandas+1.0.5.patch this:

diff --git a/node_modules/node-pandas/src/utils/getTransformedDataList.js b/node_modules/node-pandas/src/utils/getTransformedDataList.js
index 1862f4d..e871cbc 100644
--- a/node_modules/node-pandas/src/utils/getTransformedDataList.js
+++ b/node_modules/node-pandas/src/utils/getTransformedDataList.js
@@ -1,4 +1,4 @@
-const dataType = require('../utils/dataType')
+const dataType = require('../utils/datatype')
 
 function getTransformedDataList(dataList, columns) {
     let index = []
diff --git a/node_modules/node-pandas/src/utils/utils.js b/node_modules/node-pandas/src/utils/utils.js
index e09bab3..c7c4127 100644
--- a/node_modules/node-pandas/src/utils/utils.js
+++ b/node_modules/node-pandas/src/utils/utils.js
@@ -1,4 +1,4 @@
-const dataType = require('./dataType') // require('dataType') => Error: Cannot find module 'dataType'
+const dataType = require('./datatype') // require('dataType') => Error: Cannot find module 'dataType'
 const getTransformedDataList = require('./getTransformedDataList')
 const getIndicesColumns = require('./getIndicesColumns')
 const excludingColumns = require("./excludingColumns")

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.

1 participant