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

Add typescript definitions #5

Open
wants to merge 14 commits into
base: master
Choose a base branch
from
Open

Conversation

nadavsinai
Copy link

@nadavsinai nadavsinai commented Nov 20, 2017

just a simple typedef of your great lib.
allows to use in typescript environment.
if you want me to rebase I can do, I did all these commits on the online github editor ;-) and then cloned my fork...
you can also merge with squash to get one commit into your history

i've also added a commit to relax node engine constrain since it gave me trouble when installing with Yarn
tag version 1.3,0

I've now also added a fix for when this is installed globally on windows and the global npm packages are within \program files\npm... (for instance -really anything with a space) the powershell script would not run as the path was not quoted and it would try only to run the part with the c:\program ....
tag 1.3.1

since I need this... I added the commits made by @brianantonelli (Cox-Automotive ) and tagged now with
tag 1.3..2

@nadavsinai
Copy link
Author

your CI config needs a touch... checking node 4 is a little outdated I would say, your tests run fine locally

Nadav Sinai added 3 commits December 6, 2017 10:50
…to Cox-Automotive-master

# Conflicts:
#	lib/run-powershell.js
#	src/run-powershell.js
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