axodotdev / axodotdev/cargo-dist

Deps: Is console.table necessary?

Open
#2,224 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Rust
Stars
2.1k
Forks
149
Avg merge
1d 11h
Merged PRs (30d)
32

Description

[In the npm installer `binary.js` file, it references console.table](https://github.com/axodotdev/cargo-dist/blob/c53b7d91e9daf36bbedf8a37a60a3e9863142862/cargo-dist/templates/installer/npm/binary.js#L3)

This seems to be unused since there are no calls to console.table beyond this import

Beyond that, the minimum node version is marked as 14, and [console.table has been shipped with node since node 10](https://nodejs.org/en/blog/release/v10.0.0)

Contributor guide

Open the contributing guide

Research direction

Open cargo-dist/templates/installer/npm/binary.js at the referenced line and check the surrounding imports and references to console.table. Confirm that no calls use it and verify that the npm installer remains valid without the unused reference; done means the file no longer contains an unnecessary console.table reference.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript, node.js
Domain
release
Issue type
Refactor
Difficulty
1/5
Estimated time
Under an hour
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
58/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.