MattMcFarland / vscode-maniascript

VSCode Grammar / Autocompletion support for ManiaScript;
https://marketplace.visualstudio.com/items?itemName=mmcfarland.vscode-maniascript#review-details
MIT License
14 stars 3 forks source link

Unable to run the code #1

Closed Noomaok closed 6 years ago

Noomaok commented 6 years ago

Hello,

I have recently been interested in the maniascript language and i saw you had made an extension of Visual Studio Code. However, there have been some update to this language, so i would like to make an update for this program.

I've forked this repository to try and make some changes, but i got stuck with a problem when i try to compile the project. I got this error : npm-run-failed

I've import and install all the node modules needed in the different js files and i still get this error. I am quite new to all of this stuff, so maybe i have done something wrong. Also, i am on windows 10, node version is v8.11.1 and npm version is 5.6.0. I wish you can help me :)

If you need more details or anything else, feel free to ask.

MattMcFarland commented 6 years ago

Hey! looks like its windows being well, windows :P

Please download git bash for windows, and run the commands in bash, this should fix the problem!!

Node / npm doesnt work very well in cmd or powershell unfortunately.

On Sat, Apr 7, 2018 at 9:56 AM, Noomaok notifications@github.com wrote:

Hello,

I have recently been interested in the maniascript language and i saw you had made an extension of Visual Studio Code. However, there have been some update to this language, so i would like to make an update for this program.

I've forked this repository to try and make some changes, but i got stuck with a problem when i try to compile the project. I got this error : [image: npm-run-failed] https://user-images.githubusercontent.com/25253796/38455666-b33d5ed4-3a7b-11e8-9788-2ae49ab93934.png

I've import and install all the node modules needed in the different js files and i still get this error. I am quite new to all of this stuff, so maybe i have done something wrong. Also, i am on windows 10, node version is v8.11.1 and npm version is 5.6.0. I wish you can help me :)

If you need more details or anything else, feel free to ask.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/MattMcFarland/vscode-maniascript/issues/1, or mute the thread https://github.com/notifications/unsubscribe-auth/AAmDubC3-7lmOrCJC68NfAr9lY7ddW0wks5tmMWegaJpZM4TLFzI .

--

Matt McFarland

Software Engineer | @docodemore https://twitter.com/docodemore | LinkedIn http://www.linkedin.com/pub/matthew-mcfarland/6a/b9a/135/en | Mobile: 513.638.8685 | Office: 937.865.6800 xt: 125010

Noomaok commented 6 years ago

Thank you very much, indeed windows do strange things :p I have just one question left :

While adding classes to the completions.json file, I noticed that some methods have sometimes mutliples way to be uses (with different parameters) like so : multiple_parameters Is it possible to add them in this structure ?

MattMcFarland commented 6 years ago

Yes you should be able to do this. This is called "function overloading" - you might have to do some research on adding this support

On Sat, Apr 7, 2018 at 1:46 PM, Noomaok notifications@github.com wrote:

Thank you very much, indeed windows do strange things :p I have just one question left :

While adding classes to the completions.json file, I noticed that some methods have sometimes mutliples way to be uses (with different parameters) like so : [image: multiple_parameters] https://user-images.githubusercontent.com/25253796/38458192-a2a95f80-3a9b-11e8-9fdb-26c2c220bba3.png Is it possible to add them in this structure ?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/MattMcFarland/vscode-maniascript/issues/1#issuecomment-379487016, or mute the thread https://github.com/notifications/unsubscribe-auth/AAmDuQKG1mRs8Bnt86QDuvT9r0rcgMOIks5tmPuPgaJpZM4TLFzI .

--

Matt McFarland

Software Engineer | @docodemore https://twitter.com/docodemore | LinkedIn http://www.linkedin.com/pub/matthew-mcfarland/6a/b9a/135/en | Mobile: 513.638.8685 | Office: 937.865.6800 xt: 125010

Noomaok commented 6 years ago

Ok, thank you ! I am going to work on that You can close the issue if you want :)