Skip to content

Conversation

@wavebeem
Copy link
Contributor

No description provided.

@wavebeem
Copy link
Contributor Author

It looks like the tests failed, but all I changed was the README, so I assume I didn't cause that?

README.md Outdated
npm install
npm start
> npm install
> npm install node-pty
Copy link
Member

Choose a reason for hiding this comment

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

You shouldn't need to do this as it's done in npm install

@wavebeem
Copy link
Contributor Author

Oh, I see, I think that was just because my build stuff wasn't set up right, and that was the only module I needed to reinstall. Removing.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 64.637% when pulling 96d2a2b on wavebeem:issue/562 into cf4da6c on sourcelair:master.

@Tyriar
Copy link
Member

Tyriar commented Feb 27, 2017

The test failure is related to TypeScript 2.2, I made a commit to fix the issue.

Thanks @wavebeem 😄

@Tyriar Tyriar merged commit 3e5fcf7 into xtermjs:master Feb 27, 2017
@wavebeem wavebeem deleted the issue/562 branch February 27, 2017 20:04
@wavebeem
Copy link
Contributor Author

Thanks for your help and the sweet project! 👍

@Tyriar Tyriar modified the milestone: 2.4.0 Mar 3, 2017
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