awslabs / llrt

LLRT (Low Latency Runtime) is an experimental, lightweight JavaScript runtime designed to address the growing demand for fast and efficient Serverless applications.
Apache License 2.0
7.73k stars 341 forks source link

fix: make check broken #408

Closed nabetti1720 closed 3 weeks ago

nabetti1720 commented 3 weeks ago

Issue # (if available)

Related #406

Description of changes

Fix make check no longer passes.

% make check
cargo clippy --all-targets --all-features -- -D warnings
   Compiling llrt_core v0.1.14-beta (/Users/shinya/Workspaces/llrt/llrt_core)
error: the item `console` is imported redundantly
   --> llrt_core/src/runtime_client.rs:470:13
    |
6   | use crate::modules::console;
    |     ----------------------- the item `console` is already imported here
...
470 |         use crate::modules::console;
    |             ^^^^^^^^^^^^^^^^^^^^^^^
    |
    = note: `-D unused-imports` implied by `-D warnings`
    = help: to override `-D warnings` add `#[allow(unused_imports)]`

error: the item `Rest` is imported redundantly
   --> llrt_core/src/runtime_client.rs:471:13
    |
23  | use rquickjs::function::{Rest, This};
    |                          ---- the item `Rest` is already imported here
...
471 |         use rquickjs::function::Rest;
    |             ^^^^^^^^^^^^^^^^^^^^^^^^

error: could not compile `llrt_core` (lib) due to 2 previous errors
make: *** [check] Error 101

Checklist

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.