Skip to content
This repository has been archived by the owner on Aug 11, 2022. It is now read-only.

Update cli-table2 to cli-table3 #20955

Closed
wants to merge 1 commit into from
Closed

Conversation

Turbo87
Copy link

@Turbo87 Turbo87 commented Jun 11, 2018

This PR updates the cli-table2 dependency to cli-table3, which fixes one of the two remaining npm audit warnings :)

npm/npm-audit-report#24 fixes the other warning.

cli-table2 (like cli-table itself) is no longer maintained. In jamestalmage/cli-table2#43 a couple of people have offered to take over maintenance but the current maintainer did not respond so as a result the project was forked to https://github.com/cli-table/cli-table3.

@zkat
Copy link
Contributor

zkat commented Jul 10, 2018

I did this upgrade by hand on this repo (accepted the PR in npm-audit-report). We're all set here. Thanks! So happy to get rid of All Of lodash™. Saved like 300kb+ from the tarball itself (much more after extracting).

@zkat zkat closed this Jul 10, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants