Skip to content

[2.10] remove readies from memcheck#6907

Merged
BenGoldberger merged 1 commit into2.10from
backport-6868-to-2.10
Sep 29, 2025
Merged

[2.10] remove readies from memcheck#6907
BenGoldberger merged 1 commit into2.10from
backport-6868-to-2.10

Conversation

@redisearch-backport-pull-request
Copy link
Contributor

@redisearch-backport-pull-request redisearch-backport-pull-request bot commented Sep 28, 2025

Description

Backport of #6868 to 2.10.


Note

Remove readies sourcing from sbin/memcheck-summary and inline color escape variables.

  • Scripts:
    • Update sbin/memcheck-summary:
      • Remove export READIES=$ROOT/deps/readies and sourcing of $READIES/shibumi/defs.
      • Inline color variables: RED, LIGHTRED, NOCOLOR.

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

* remove readies from memcheck

* add colors

* red

(cherry picked from commit f1df980)
@codecov
Copy link

codecov bot commented Sep 28, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.29%. Comparing base (b38f839) to head (09bf6c2).
⚠️ Report is 4 commits behind head on 2.10.

Additional details and impacted files
@@            Coverage Diff             @@
##             2.10    #6907      +/-   ##
==========================================
- Coverage   89.31%   89.29%   -0.03%     
==========================================
  Files         207      207              
  Lines       36165    36165              
==========================================
- Hits        32302    32293       -9     
- Misses       3863     3872       +9     
Flag Coverage Δ
flow 83.81% <ø> (-0.15%) ⬇️
unit 42.72% <ø> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@redisearch-backport-pull-request redisearch-backport-pull-request bot added this pull request to the merge queue Sep 28, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Sep 28, 2025
@BenGoldberger BenGoldberger added this pull request to the merge queue Sep 29, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Sep 29, 2025
@BenGoldberger BenGoldberger added this pull request to the merge queue Sep 29, 2025
Merged via the queue into 2.10 with commit 026a197 Sep 29, 2025
13 checks passed
@BenGoldberger BenGoldberger deleted the backport-6868-to-2.10 branch September 29, 2025 12:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant