-
Notifications
You must be signed in to change notification settings - Fork 75
tsup #644
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
tsup #644
Changes from 2 commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
This file was deleted.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -5,6 +5,5 @@ module.exports = { | |
| "reject": [ | ||
| "chai", // Moved to es6 | ||
| "@types/chai", // Should match chai, | ||
| "rimraf", // Requires node 20 | ||
|
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. We're node 20+ now, no need to keep rimraf back at a previous version. Chai still needs to be held back until either we switch to modules or I try testing it with require(mjs), which should work in node 20.
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. require(mjs) should work, except for jest. TODO: remove Jest, which is a problem for another day. |
||
| ], | ||
| }; | ||
|
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Moved to files: in package.json, so we can stop maintaining this brittle file. |
This file was deleted.
|
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Left over from old express server. Now tested from docs with eleventy as the server. |
This file was deleted.
|
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Left over from old express server. Now tested from docs with eleventy as the server. |
This file was deleted.
|
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Left over from old express server. Now tested from docs with eleventy as the server. |
This file was deleted.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,6 +1,6 @@ | ||
| "use strict"; | ||
|
|
||
| const peg = require("../lib/peg"); | ||
| const peg = globalThis.peggy || require("../lib/peg"); | ||
|
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Hack so that we can use the global peggy in benchmarks, loaded before this script is run. Dramatically decreases the size of the benchmark, because the entirety of Peggy doesn't have to be bundled in. Increases my confidence that peggy.min.js is correct, since we're actually testing it a little. |
||
|
|
||
| const Runner = { | ||
| run(benchmarks, runCount, options, callbacks) { | ||
|
|
||
|
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Left over from old express server. Now tested from docs with eleventy as the server. |
This file was deleted.
|
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Left over from old express server. Now tested from docs with eleventy as the server. |
This file was deleted.
|
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Left over from old express server. Now tested from docs with eleventy as the server. |
This file was deleted.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This file is left over from a previous web server that we used for docs.