New feature 25 "Deterministic Finality and Ride V9" added. #1834

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.

Sign up for GitHub

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

Open

alexeykiselev wants to merge 87 commits into master from determenistic-finality-feature

Open

New feature 25 "Deterministic Finality and Ride V9" added. #1834

Merge branch 'master' into determenistic-finality-feature

GitHub Advanced Security / Semgrep OSS succeeded Apr 23, 2026 in 2s

3 new alerts

New alerts in code changed by this pull request

3 warnings

Alerts not introduced by this pull request might have been detected because the code changes were too large.

See annotations below for details.

View all branch alerts.

Annotations

Check warning on line 95 in itests/config/miningtype_string.go

Code scanning / Semgrep OSS

Semgrep Finding: semgrep.rules.if-incorrect-nil-err-return Warning

WARNING: A local variable 'err' is checked for nil, but a different variable is returned.
Ensure that the returned variable is the one that was checked or properly wrapped!

Check warning on line 95 in itests/config/miningtype_string.go

Code scanning / Semgrep OSS

Semgrep Finding: semgrep.rules.if-inplace-func-incorrect-nil-err-return Warning

WARNING: A local variable 'err' is checked for nil, but a different variable is returned.
Ensure that the returned variable is the one that was checked or properly wrapped!

Check warning on line 7 in pkg/ride/ride_constructors_test.go

Code scanning / Semgrep OSS

Semgrep Finding: go.lang.security.audit.crypto.math_random.math-random-used Warning test

Do not use math/rand. Use crypto/rand instead.