Skip to content
This repository was archived by the owner on Apr 9, 2024. It is now read-only.

Update npm command to support version 9#116

Closed
andyreagan wants to merge 1 commit into
altair-viz:masterfrom
andyreagan:patch-1
Closed

Update npm command to support version 9#116
andyreagan wants to merge 1 commit into
altair-viz:masterfrom
andyreagan:patch-1

Conversation

@andyreagan
Copy link
Copy Markdown

NPM drops support for the bin command in version 9 (ref here), while root is backwards compatible: ref.

NPM drops support for the `bin` command in version 9 ([ref here](https://docs.npmjs.com/cli/v7/commands/npm-bin?v=true)), while `root` is backwards compatible: [ref](https://docs.npmjs.com/cli/v9/commands/npm-root?v=true).
@andyreagan
Copy link
Copy Markdown
Author

@jakevdp I think this should be a pretty simple 1-line fix to merge in?

I just checked the issues tab, and this closes #113 .

@fleimgruber
Copy link
Copy Markdown

AFAICS, the failing CI checks are not due to changes in this PR, but some issues on the CI env/config.

@Lundez
Copy link
Copy Markdown

Lundez commented Jun 20, 2023

The anaconda distribution can be solved by using nodejs=18.13.0 as a forced dependency.

Anaconda had it auto-updated which later used npm9+ that fails...

@joelostblom
Copy link
Copy Markdown
Member

Thanks for you PR @andyreagan. As you have noticed, there hasn't been any development in this repo for a while. Since Altair 5.2, the functionality of Altair Saver is now available in Altair via the vl-convert package. Most of the functionality has been available since 5.0, and the main addition in 5.2 was PDF export. See the docs on how to save charts for more details.

We are going to archive this repo, so I'm closing all the open issues and PRs before doing so. Try out the new options for saving charts mentioned above and if you run into issues, please open an issue directly in the altair or vl-convert repo.

@andyreagan
Copy link
Copy Markdown
Author

Thanks for the very thoughtful follow up here! I'll be using vl-convert in the future :)

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.

4 participants