Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TS 2.9 follow up #3839

Merged
merged 2 commits into from
Jun 18, 2018
Merged

TS 2.9 follow up #3839

merged 2 commits into from
Jun 18, 2018

Conversation

domoritz
Copy link
Member

@domoritz domoritz commented Jun 1, 2018

@domoritz domoritz force-pushed the dom/ts2.9-follow-up branch 3 times, most recently from 37c28d7 to faf34cb Compare June 2, 2018 00:01
@domoritz domoritz force-pushed the dom/ts2.9-follow-up branch 2 times, most recently from 0a4ae33 to 440a25e Compare June 2, 2018 01:00
@jasongrout
Copy link

jasongrout commented Jun 5, 2018

I had another issue upgrading to vega-lite 2.5.1 that I thought was related to the ts 2.9 upgrade, but decided it really was another issue: #3855

@domoritz domoritz force-pushed the dom/ts2.9-follow-up branch 3 times, most recently from 36fbfe0 to cb90ea8 Compare June 14, 2018 23:48
@domoritz
Copy link
Member Author

This change simplifies the package.json and also improves our sourcemaps.

@domoritz domoritz force-pushed the dom/ts2.9-follow-up branch from cb90ea8 to 6373e23 Compare June 15, 2018 06:38
@@ -45,7 +45,7 @@
"renameschema": "scripts/rename-schema.sh",
"presite": "npm run prebuild && npm run data && npm run build:site && npm run build:toc && npm run build:versions && scripts/create-example-pages",
"site": "bundle exec jekyll serve --incremental",
"lint": "tslint -p .",
"lint": "tslint -p . -e 'package.json'",
Copy link
Member Author

Choose a reason for hiding this comment

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

This is to get around a bug in tslint.

Copy link
Member Author

Choose a reason for hiding this comment

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

@domoritz
Copy link
Member Author

Ready @kanitw

@domoritz domoritz requested a review from kanitw June 18, 2018 21:09
@domoritz domoritz merged commit acecc09 into master Jun 18, 2018
@domoritz domoritz deleted the dom/ts2.9-follow-up branch June 18, 2018 21:19
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