search

nginx / nginx.org

Sources for the NGINX website and documentation
https://nginx.org
Other
48 stars 18 forks source link

Added note about nginx for Windows beta version. #48

Open y82 opened 2 months ago

oxpa commented 1 month ago

IMO, "beta version" assumes that the software will be released into a production version at some point. As we are not going to release nginx for windows any time soon, I suppose something like below would be better.

nginx is available for windows for local development and testing purposes. It is not meant to be used in a production environments but rather as an aid for windows based developers.

This wording clearly states the purpose of the windows build and doesn't promise any changes into it.

lcrilly commented 1 month ago

FWIW, the Docker Hub readme says this

[nginx] also has a proof of concept port for Microsoft Windows.

pluknet commented 1 month ago

the page already contains all the necessary parts, note:

pluknet commented 1 month ago

the page already contains all the necessary parts, no need to further adjust it:

Due to this and some other known issues version of nginx for Windows is considered to be a beta version.

lcrilly commented 1 month ago

It is not clear enough and many people seek production support for Windows. If nothing else, this page would still benefit from having the "beta version" phrase moved to the top of the article.

@y82 should be given the opportunity to respond before this is closed or resubmitted.

pluknet commented 1 month ago

It is not clear actually how many people failed to read the existing (and emphasized) text, that's not our fault. Changing it at someone's whim is a bad practice.

To reiterate: IMHO, the beta status is clearly enough articulated. Given the PR name, this is also a bad change: a beta version status is already present, there's nothing to "add".

If ever change anything, we could move the existing "beta version" note to make it the 1st sentence, though I'd leave it as is.

For sure, it's up to @y82 to decide.