Added support for windows Unistall/DisplayVersion

[demetris-manikas]

Very small patch to allow for version property in the nsi templates.
Let me know what you think.
Cheers

[mention-bot]

By analyzing the blame information on this pull request, we identified @stefanjudis and @vxsx to be potential reviewers.

[stefanjudis]

Thanks.

Fine for me - but it's missing the readme mention about that?

Also what happens if version is not defined - is it still working with an empty string then?

[demetris-manikas]

Updated the README.md but you probably have to rephrase it.
If version is not defined APP_VERSION is just an empty string

[stefanjudis]

@demetris-manikas
Now that I'm thinking of it defaulting to version in package.json would be nice maybe?

[demetris-manikas]

I played with the idea but it needed more effort. Where is the package.json located? In my case it 's not even in the same dir as config.json. So things get complicated. Setting the path to the package.json would be a way. Should that be a global or a win param? These question made me go for the simple solution. Anyway it is very easy to produce a config.json containing the appropriate data.

[stefanjudis]

Okay - I see. Fair enough.