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

Updates to latest tinygo and coraza-wasilibs #231

Merged
merged 1 commit into from
Oct 2, 2023

Conversation

M4tteoP
Copy link
Member

@M4tteoP M4tteoP commented Oct 2, 2023

tinygo 0.30.0 is out and our tinygo version check is very strict. The update looks okay.
Updated also coraza-wasilibs that includes go-re2 latest release.

@@ -20,7 +20,7 @@ import (
)

var minGoVersion = "1.20"
var tinygoMinorVersion = "0.29"
var tinygoMinorVersion = "0.30"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

BTW I have been thinking compatibility has been good for the past 3-4 releases I think. We could probably make this check a minimum similar to the Go version check.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sounds like something we could do, any advice @deadprogram?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I opened #232 if we feel we can relax tinygo version check

@M4tteoP M4tteoP merged commit 9368451 into corazawaf:main Oct 2, 2023
@M4tteoP M4tteoP deleted the update_tinygo_re2 branch October 4, 2023 15:42
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

Successfully merging this pull request may close these issues.

3 participants