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

Upload database action failing #288

Closed
JamieMagee opened this issue Apr 6, 2023 · 0 comments · Fixed by #298
Closed

Upload database action failing #288

JamieMagee opened this issue Apr 6, 2023 · 0 comments · Fixed by #298

Comments

@JamieMagee
Copy link
Contributor

The Upload database has been failing since 8e99a29 on the the 1st April. The error is:

/home/runner/work/osv-offline/osv-offline/node_modules/ts-node/dist/util.js:62
    return value.replace(backslashRegExp, directorySeparator);
                 ^

TypeError: value.replace is not a function
    at normalizeSlashes (/home/runner/work/osv-offline/osv-offline/node_modules/ts-node/dist/util.js:62:18)
    at Object.getExtendsConfigPath (/home/runner/work/osv-offline/osv-offline/node_modules/ts-node/dist/ts-internals.js:24:54)
    at readConfig (/home/runner/work/osv-offline/osv-offline/node_modules/ts-node/dist/configuration.js:127:64)
    at findAndReadConfig (/home/runner/work/osv-offline/osv-offline/node_modules/ts-node/dist/configuration.js:50:84)
    at phase3 (/home/runner/work/osv-offline/osv-offline/node_modules/ts-node/dist/bin.js:254:67)
    at bootstrap (/home/runner/work/osv-offline/osv-offline/node_modules/ts-node/dist/bin.js:47:30)
    at main (/home/runner/work/osv-offline/osv-offline/node_modules/ts-node/dist/bin.js:33:12)
    at Object.<anonymous> (/home/runner/work/osv-offline/osv-offline/node_modules/ts-node/dist/bin.js:579:5)
    at Module._compile (node:internal/modules/cjs/loader:1254:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1308:10)

and is due to ts-node's lack of support for TypeScript 5.0's extends array.

This was resolved in TypeStrong/ts-node#1958 but is not yet available in a release. The two options are to:

  1. Rollback to a single extends
  2. Use ts-node from GitHub instead of npm
JamieMagee added a commit that referenced this issue Apr 16, 2023
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 a pull request may close this issue.

1 participant