Support running via eval/require as well as vm2

[orta]

Gets relative paths working correctly via vm2 - deprecates #359

Updates us to a vm2 which has an index.d.ts shipped patriksimek/vm2#92

Also makes the dangerfile use the custom tsconfig - which means it gets the include function.

[DangerCI]

New dependencies added: require-from-string.

require-from-string

Author: Vsevolod Strukchinsky

Description: Require module from string

Homepage: https://github.com/floatdrop/require-from-string#readme

Created	about 2 years ago
Last Updated	28 days ago
License	MIT
Maintainers	1
Releases	8
Direct Dependencies

README

require-from-string

Load module from string in Node.

Install

$ npm install --save require-from-string

Usage

var requireFromString = require('require-from-string');

requireFromString('module.exports = 1');
//=> 1

API

requireFromString(code, [filename], [options])

code

Required
Type: string

Module code.

filename

Type: string
Default: ''

Optional filename.

options

Type: object

appendPaths

Type: Array

List of paths, that will be appended to module paths. Useful, when you want
to be able require modules from these paths.

prependPaths

Type: Array

Same as appendPath, but paths will be prepended.

License

MIT © Vsevolod Strukchinsky

yarn why require-from-string output

• Has been hoisted to "require-from-string"
• This module exists because it's specified in "dependencies".
• Disk size without dependencies: "16kB"
• Disk size with unique dependencies: "16kB"
• Disk size with transitive dependencies: "16kB"
• Number of shared dependencies: 0

Generated by 🚫 dangerJS

[orta]

I was testing by using ts-node, which I bet was pre-compiling it in some form for me, will need to test this by using the built versions. Can't write a test with Jest either as it will compile also. 🍡

[orta]

OK, I have a clue

When running via the eval, the file for which all relative imports are working from is of course the runner. Not the cwd for the dangerfile.