axodotdev / axodotdev/cargo-dist
Deps: Is console.table necessary?
- 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
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