Skip to content

[2.8] remove readies from memcheck#6905

Merged
BenGoldberger merged 1 commit into2.8from
backport-6868-to-2.8
Sep 30, 2025
Merged

[2.8] remove readies from memcheck#6905
BenGoldberger merged 1 commit into2.8from
backport-6868-to-2.8

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.8.


Note

Removes readies dependency from sbin/memcheck-summary and inlines ANSI color variables, updating colorized output usage.

  • sbin/memcheck-summary:
    • Remove export READIES and sourcing of shibumi/defs from deps/readies.
    • Add inline ANSI color vars: RED, LIGHTRED, NOCOLOR.
    • Update output to use NOCOLOR resets and colorized messages in valgrind_check.

Written by Cursor Bugbot for commit b1d19f2. 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 87.49%. Comparing base (2cf1600) to head (b1d19f2).
⚠️ Report is 3 commits behind head on 2.8.

Additional details and impacted files
@@            Coverage Diff             @@
##              2.8    #6905      +/-   ##
==========================================
- Coverage   87.50%   87.49%   -0.01%     
==========================================
  Files         203      203              
  Lines       35634    35634              
==========================================
- Hits        31181    31178       -3     
- Misses       4453     4456       +3     
Flag Coverage Δ
flow 81.81% <ø> (-0.23%) ⬇️
unit 42.75% <ø> (ø)

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
@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
@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
@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 30, 2025
Merged via the queue into 2.8 with commit a01045c Sep 30, 2025
13 checks passed
@BenGoldberger BenGoldberger deleted the backport-6868-to-2.8 branch September 30, 2025 07:20
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