search

neutralinojs / neutralinojs-cli

neu cli for Neutralinojs
https://neutralino.js.org/docs/cli/neu-cli
MIT License
91 stars 56 forks source link

Show warning for outdated CLI version #260

Closed Sadaf-A closed 2 months ago

Sadaf-A commented 3 months ago

Resolves: #210

This PR adds a utility function to show a warning message when the CLI version is outdated

warn(1)

viralgupta commented 3 months ago

Hi @Sadaf-A, do you think we should use node:http module instead of axios? it would help us to reduce the number of deps.

Sadaf-A commented 3 months ago

Hi @Sadaf-A, do you think we should use node:http module instead of axios? it would help us to reduce the number of deps.

Hey, I decided to the use fetch. We can not use http module since the url is https. Even though we do have an https module it would make the code more verbose. Therefore, I decided to use fetch it would do both the jobs of keeping the code clean and as well as remove the extra dependency. Thanks for the suggestion!

shalithasuranga commented 2 months ago

Used npm view (https://github.com/neutralinojs/neutralinojs-cli/commit/1b4b2d7201668f69cdf7a41fe45dfef79db5d865) and merged. Thanks so much for sending this PR :tada: