Skip to content

chore(go): use -compat=1.24 in scripts/release.sh#3754

Merged
ndyakov merged 1 commit into
redis:masterfrom
cxljs:scrpits-go-version
Mar 30, 2026
Merged

chore(go): use -compat=1.24 in scripts/release.sh#3754
ndyakov merged 1 commit into
redis:masterfrom
cxljs:scrpits-go-version

Conversation

@cxljs

@cxljs cxljs commented Mar 29, 2026

Copy link
Copy Markdown
Contributor

Note

Low Risk
Low risk: updates the release automation to run go mod tidy with a newer compatibility version, which may slightly alter generated go.mod/go.sum output during releases but doesn’t change runtime code paths.

Overview
Updates the release script to use go mod tidy -compat=1.24 instead of -compat=1.21 when preparing a release, aligning module tidying output with Go 1.24 compatibility expectations.

Written by Cursor Bugbot for commit 355c642. This will update automatically on new commits. Configure here.

Signed-off-by: Xiaolong Chen <fukua95@gmail.com>

@ndyakov ndyakov left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

thanks for cleaning this up, we do not use this script nowadays, but it is good to be ready if we switch back to it.

@ndyakov ndyakov merged commit 0a45121 into redis:master Mar 30, 2026
32 of 33 checks passed
romanpovol pushed a commit to romanpovol/go-redis that referenced this pull request Apr 8, 2026
Signed-off-by: Xiaolong Chen <fukua95@gmail.com>
ndyakov added a commit that referenced this pull request Apr 9, 2026
* feat(vectorset): WIP

* feat(vectorsets): WITHATTRIBS & VIsMember

* fix(vectorsets): fix tests

* fix(vectorsets): improve doctest/

* feat(vectorsets): add test with nil attrs

* fix(vectorsets): typo Epsilon

* fix: review issues

* chore(go): use -compat=1.24 (#3754)

Signed-off-by: Xiaolong Chen <fukua95@gmail.com>

* chore(maps): Use `maps.Keys`, `slices.Collect` and `slices.Contains` instead of custom functions (#3758)

* Optimize: Use modern Go standard library (maps.Keys, slices.Collect, slices.Contains) instead of custom functions

* format

* chore(deps): bump codecov/codecov-action from 5 to 6 (#3757)

Bumps [codecov/codecov-action](https://github.com/codecov/codecov-action) from 5 to 6.
- [Release notes](https://github.com/codecov/codecov-action/releases)
- [Changelog](https://github.com/codecov/codecov-action/blob/main/CHANGELOG.md)
- [Commits](codecov/codecov-action@v5...v6)

---
updated-dependencies:
- dependency-name: codecov/codecov-action
  dependency-version: '6'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* chore(ring): inline rendezvous hashing and remove external dependency (#3762)

* ring: remove external rendezvous dependency by inlining implementation

Replace usage of github.com/dgryski/go-rendezvous with an internal
implementation to eliminate the external dependency and avoid licence
propagation concerns.

The implementation preserves the exact hashing algorithm and behaviour:
- same hash combination: khash ^ nhash
- same mixing function (xorshift* variant)
- same node ordering semantics

No functional changes intended.

* Refactor TestRendezvous to remove randomString function

Replaced function with inline string generation for test cases.

* test: refactor RendezvousHash tests to use Ginkgo and Gomega

* test: improve RendezvousHash test for per-instance determinism and distribution balance

* test: enhance distribution verification in RendezvousHash tests

---------

Signed-off-by: Xiaolong Chen <fukua95@gmail.com>
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: cxljs <fukua95@gmail.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Bigsk <i@ianxia.com>
Co-authored-by: Nedyalko Dyakov <1547186+ndyakov@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants