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
feat: bun --print
#9358
feat: bun --print
#9358
Conversation
❌ @dylan-conway 4 files with test failures on bun-darwin-aarch64: |
❌ @dylan-conway 3 files with test failures on bun-darwin-x64: |
❌🪟 @dylan-conway, there are 14 test regressions on Windows x86_64
|
Instead of making GlobalObjectMethodTable non-const, couldn't we make Zig::GlobalObject a non-final class and then have Zig::EvalGlobalObject which would have the slightly different code path? |
❌ @dylan-conway 5 files with test failures on linux-x64-baseline: |
❌ @dylan-conway 6 files with test failures on linux-x64:
|
@@ -537,7 +538,13 @@ pub const Arguments = struct { | |||
} | |||
|
|||
if (args.option("--port")) |port_str| { | |||
opts.port = std.fmt.parseInt(u16, port_str, 10) catch return error.InvalidPort; | |||
if (comptime cmd == .RunAsNodeCommand) { | |||
// TODO: prevent `node --port <script>` from working |
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.
heh nice. i will keep this in mind for the CLI rewrite
What does this PR do?
Adds
--print
to runtime flags. This flag is the same asnode --print <script>
, printing the result of the script, and also works with esm.fixes #5698
How did you verify your code works?
new and existing tests