-
Notifications
You must be signed in to change notification settings - Fork 8
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
use sync.OnceValue for various regular expressions, require go1.21 #15
base: main
Are you sure you want to change the base?
Commits on Jul 15, 2024
-
gha: update actions/checkout@v4
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Configuration menu - View commit details
-
Copy full SHA for 08b97d2 - Browse repository at this point
Copy the full SHA 08b97d2View commit details -
gha: update actions/setup-go@v5
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Configuration menu - View commit details
-
Copy full SHA for 5291d10 - Browse repository at this point
Copy the full SHA 5291d10View commit details -
gha: update golangci/golangci-lint-action@v6
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Configuration menu - View commit details
-
Copy full SHA for 92b1472 - Browse repository at this point
Copy the full SHA 92b1472View commit details -
gha: update codecov/codecov-action@v4
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Configuration menu - View commit details
-
Copy full SHA for 2ab2cce - Browse repository at this point
Copy the full SHA 2ab2cceView commit details -
gha: update golangci-lint to v1.59
Error: fuzz_test.go:11:14: unused-parameter: parameter 't' seems to be unused, consider removing or renaming it as _ (revive) f.Fuzz(func(t *testing.T, data string) { ^ Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Configuration menu - View commit details
-
Copy full SHA for 0c5e42c - Browse repository at this point
Copy the full SHA 0c5e42cView commit details -
gha: update to go1.21.x, go1.22.x
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Configuration menu - View commit details
-
Copy full SHA for 343e590 - Browse repository at this point
Copy the full SHA 343e590View commit details -
use sync.OnceValue for various regular expressions, require go1.21
Using regex.MustCompile consumes a significant amount of memory when importing the package, even if those regular expressions are not used. This changes compiling the regular expressions to use a sync.OnceValue so that they're only compiled the first time they're used. There are various regular expressions remaining that are still compiled on import, but these are exported, so changing them to a sync.OnceValue would be a breaking change; we can still decide to do so, but leaving that for a follow-up. It's worth noting that sync.OnceValue requires go1.21 or up, so raising the minimum version accordingly. Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Configuration menu - View commit details
-
Copy full SHA for 4ca1403 - Browse repository at this point
Copy the full SHA 4ca1403View commit details