neutralinojs / neutralinojs-cli

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

Update hot reload patch wording #221

Closed KTibow closed 9 months ago

KTibow commented 9 months ago

@shalithasuranga my reasoning:

is there a better term instead that you would use?

shalithasuranga commented 9 months ago

Thanks for the explanation. Mm. yeah, we can improve this term seems. I think we named it as "hot reload patch" because it enables HMR support for a "Neutralinojs app". I mentioned as above since we use "Neutralinojs" also with "neu:" prefix which looks a bit redundant.

Maybe we can use one of the following terms I guess considering your suggestion:

Or, do you have another suggesion that gives the meaning of above terms? Please update the PR with a better term without code formatting changes. We can re-open this and merge I think :tada:

Thanls :tada:

KTibow commented 9 months ago

I'll make a new PR because it's easier that way (would rather not reset this branch + its impossible for me to reopen this PR)