Skip to content

vegaLite not vl for symbol name#20

Open
jamiepratt wants to merge 1 commit intothheller:masterfrom
jamiepratt:master
Open

vegaLite not vl for symbol name#20
jamiepratt wants to merge 1 commit intothheller:masterfrom
jamiepratt:master

Conversation

@jamiepratt
Copy link
Copy Markdown

I believe that vegaLite is the correct symbol name not vl.

@thheller
Copy link
Copy Markdown
Owner

Which CLJS library expects this? If you are using this in your own code you should require this directly and now use cljsjs shims.

@jamiepratt
Copy link
Copy Markdown
Author

Hanami expects it:

jsa-aerial/hanami@881a2e4

It seems like actually there was a change in the vega-lite library at some point from vl to vegaLite.

@thheller
Copy link
Copy Markdown
Owner

I do not know if there is any other library out there using vl, so the only reasonable thing to do here is to export both.

@jamiepratt
Copy link
Copy Markdown
Author

I'll file a new pull request, hang on.

@thheller
Copy link
Copy Markdown
Owner

Note that you can just do this in your own code. Doing this in shadow-cljsjs really isn't required.

@jamiepratt
Copy link
Copy Markdown
Author

Note that you can just do this in your own code. Doing this in shadow-cljsjs really isn't required.

I made the suggested change to the pull request. This might affect other libraries that will now be expecting the 'vegaLite' symbol to be available from the latest cljsjs library so perhaps it makes sense to make the change in this repo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants