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

Minor typings tweaks #909

Merged
merged 2 commits into from
Aug 21, 2017
Merged

Minor typings tweaks #909

merged 2 commits into from
Aug 21, 2017

Conversation

Tyriar
Copy link
Member

@Tyriar Tyriar commented Aug 20, 2017

Follow up on #899
Part of #898

Some changes after trying to integrate types into vscode.

Tyriar added 2 commits August 20, 2017 11:43
If we don't have this then tsc will throw errors when passing strings
to the functions, seems wrong to force casting strings to use the
functions, eg. <'bar'>style
@Tyriar Tyriar added this to the 3.0.0 milestone Aug 20, 2017
@Tyriar Tyriar self-assigned this Aug 20, 2017
@Tyriar Tyriar requested a review from parisk August 20, 2017 18:58
@coveralls
Copy link

coveralls commented Aug 20, 2017

Coverage Status

Coverage remained the same at 71.773% when pulling 8bc2242 on Tyriar:898_improve_types into 7645bdd on sourcelair:v3.

Copy link
Contributor

@parisk parisk left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@Tyriar Tyriar merged commit abe3227 into xtermjs:v3 Aug 21, 2017
@Tyriar Tyriar deleted the 898_improve_types branch August 21, 2017 17:09
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.

3 participants