Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Specify supported njs version. #72

Open
hongzhidao opened this issue Dec 13, 2023 · 3 comments
Open

Specify supported njs version. #72

hongzhidao opened this issue Dec 13, 2023 · 3 comments
Assignees

Comments

@hongzhidao
Copy link

https://unit.nginx.org/scripting/

Warning

Unit 1.31+ doesn’t support pre-0.8 njs [versions](https://nginx.org/en/docs/njs/changes.html); please update.

Because of the change of njs 0.8.2, Unit doesn't support this version, it looks like we need to specify which versions we support.

@Dean-Coakley
Copy link
Contributor

@hongzhidao I'd be happy to help out with this issue.

Would updating Unit 1.31+ doesn’t support pre-0.8 njs -> Unit 1.31+ doesn’t support pre-0.8.2 njs resolve the issue? Or can you help me understand the change you'd like to see?

@hongzhidao
Copy link
Author

Hi @Dean-Coakley
Thanks. Actually, 0.8.0-0.8.2 works.
@thresheek could you help check it?

@Jcahilltorre Jcahilltorre self-assigned this Oct 24, 2024
@Jcahilltorre
Copy link
Collaborator

the current wording is

Unit 1.32.0 and later require [njs 0.8.2](https://nginx.org/en/docs/njs/changes.html).

Does this need to be updated or can this issue we closed?

cc @hongzhidao @thresheek @callahad

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants