8000 Support installed npm modules and relative require by jclem · Pull Request #135 · actions/github-script · GitHub
[go: up one dir, main page]
Skip to content

Support installed npm modules and relative require #135

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.

Sign up for GitHub

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

Merged
merged 25 commits into from
Apr 21, 2021
Merged

Support installed npm modules and relative require #135

Changes from 1 commit
Commits
Show all changes
25 commits
Select commit Hold shift + click to select a range
ddba1b1
Use non-Webpack-require in evaluated scripts
jclem Apr 21, 2021
f4e5d39
Use a proxy to support relative requires
jclem Apr 21, 2021
e853490
Run build script
jclem Apr 21, 2021
75e3a5b
Move wrapRequire to its own module
jclem Apr 21, 2021
c416f56
Pass nativeRequire, as well
jclem Apr 21, 2021
83c92d6
Rename nativeRequire to __original_require__
jclem Apr 21, 2021
3ca4cd5
Update documentation for relative require support
jclem Apr 21, 2021
c1c139b
Add a catch to support requiring installed modules
jclem Apr 21, 2021
ade5cea
Use concat for paths
jclem Apr 21, 2021
f57c84a
Try global.module
jclem Apr 21, 2021
1f8fc98
Run build
jclem Apr 21, 2021
d37f92f
Test eval
jclem Apr 21, 2021
7e8659d
Run build
jclem Apr 21, 2021
7e12bd7
Call resolve, then require
jclem Apr 21, 2021
b0e12e7
Add Webpack comment vis-a-vis eval
jclem Apr 21, 2021
01f87b6
Remove caveat about node_modules
jclem Apr 21, 2021
3ede589
Add new integration tests
jclem Apr 21, 2021
19e7914
Add integration test running
jclem Apr 21, 2021
< 8000 code class="float-right">4a93ad9
Add actions/checkout
jclem Apr 21, 2021
fb7e8f7
Add npm ci to integration test
jclem Apr 21, 2021
5ee517d
Workflow syntax error
jclem Apr 21, 2021
256da4e
Update src/wrap-require.ts
jclem Apr 21, 2021
c758586
v4.0.0
jclem Apr 21, 2021
b616178
Merge branch 'wrap-require' of https://github.com/actions/github-scri…
jclem Apr 21, 2021
2923e50
Run build script
jclem Apr 21, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Run build
  • Loading branch information
@jclem
jclem committed Apr 21, 2021
commit 1f8fc98741b80b5d8436316606e8ecd335165240
28 changes: 2 additions & 26 deletions dist/index.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2427,7 +2427,7 @@ exports.request = request;
/***/ }),

/***/ 272:
/***/ (function(module, __webpack_exports__, __webpack_require__) {
/***/ (function(__unusedmodule, __webpack_exports__, __webpack_require__) {

"use strict";
__webpack_require__.r(__webpack_exports__);
Expand Down Expand Up @@ -2455,7 +2455,6 @@ function callAsyncFunction(args, source) {
var external_path_ = __webpack_require__(622);

// CONCATENATED MODULE: ./src/wrap-require.ts
/* module decorator */ module = __webpack_require__.hmd(module);

const wrapRequire = new Proxy(require, {
apply: (target, thisArg, [moduleID]) => {
Expand All @@ -2468,7 +2467,7 @@ const wrapRequire = new Proxy(require, {
}
catch (err) {
return target.resolve(moduleID, {
paths: module.paths.concat(process.cwd())
paths: global.module.paths.concat(process.cwd())
});
}
},
Expand Down Expand Up @@ -8812,28 +8811,5 @@ function regExpEscape (s) {
/******/ };
/******/ }();
/******/
/******/ /* webpack/runtime/harmony module decorator */
/******/ !function() {
/******/ __webpack_require__.hmd = function(module) {
/******/ module = Object.create(module);
/******/ if (!module.children) module.children = [];
/******/ Object.defineProperty(module, 'loaded', {
/******/ enumerable: true,
/******/ get: function () { return module.l; }
/******/ });
/******/ Object.defineProperty(module, 'id', {
/******/ enumerable: true,
/******/ get: function () { return module.i; }
/******/ });
/******/ Object.defineProperty(module, 'exports', {
/******/ enumerable: true,
/******/ set: function () {
/******/ throw new Error('ES Modules may not assign module.exports or exports.*, Use ESM export syntax, instead: ' + module.id);
/******/ }
/******/ });
/******/ return module;
/******/ };
/******/ }();
/******/
/******/ }
);
0