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

[WIP] Resurrect victory-cli #7

Closed
wants to merge 5 commits into from
Closed

[WIP] Resurrect victory-cli #7

wants to merge 5 commits into from

Conversation

kale-stew
Copy link

In its current state, this package is not downloadable via any package manager (npx, npm, and yarn all fail to download). This PR addresses the following issues to bring it back to a functional state:

  • Updated dependencies across the board
  • Updated usage of the vastly improved (although also, now deprecated) phantom
  • Reintroducing Ci
  • More things that will crop up, I'm sure

@kale-stew kale-stew marked this pull request as draft June 3, 2020 17:41
@chrisbolin
Copy link

I'd like to suggest that we drop support for PNG and remove all librsvg dependencies. This would do a few things for us...

also, svgs are great! they are supported in all browser and scalable (they are vectors, dangit)!

@@ -24,8 +23,6 @@ program.option(
"'area', 'bar', 'line', 'scatter' or 'pie'",
"line"
);
program.option("-f, --format [format]", "'png' or 'svg'", "png");
program.option("-p, --print", "Prints chart to console (iTerm3 & .png format only!)");
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As this feature was dependent on a generated png, we no longer support printed imgs to iTerm either

@kale-stew kale-stew closed this Jan 8, 2022
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