Hey @pelotom, could you grant me and @jheer access to the npm package?

From https://vega.github.io/vega/docs/marks/line/. For example: strokeCap, strokeJoin, ...

We use these comments to generate the schema. See https://github.com/vega/vega-lite/blob/master/src/vega.schema.ts

We currently cannot compile Vega-Lite with Vega-Lib because vega-lib does not declare typings for vega. This is related to #8.

I suggest that we either merge #8 or add

declare module 'vega' {
  export * from 'vega-typings';
}

to vega-lib. I'm not sure what is better.

Source (vega docs)

In types/spec/axis.d.ts tickCount is number | SignalRef so string values are not valid.

Could you properly adjust the typings?

https://github.com/vega/vega-util already has typings. We should use them.

Also, include some test that would force us to update this repo whenever Vega got new syntax?

I've been using vega for several weeks now in Angular projects and I'm happy to see that an official typings files are being made!

Unfortunately, right now the typings are missing a couple functions which I rely on in order to interact with graphs via signals. These functions are :

// in types/runtime/index.d.ts in the class View
addSignalListener(name: string, handler: (name: string, value: any) => void): View;
removeSignalListener(name: string, handler: (name: string, value: any) => void): View;
// only the version with two parameters exists, not this one that allows to get a signal's value
signal(name: string): any;

I understand that the typings are a work in progress right now. I will be using a old version in the meantime, I just wanted to be notified when these functions are added to the typings.

Thanks for your work!

Se already have some typings for transforms in https://github.com/vega/vega-lite/blob/master/src/vega.schema.ts.

cc @pelotom

After upgrading to [email protected]:

ERROR in node_modules/vega-typings/types/spec/util.d.ts(8,50): error TS2304: Cannot find name 'Exclude'.

My app uses Angular 6 which is still at [email protected].
I guess Exclude was added in [email protected], so all apps below 2.8 will throw error.

Looks like the documentation comments here are copied from Vega-Lite, which often has different default from Vega.