Skip to content

remove promisfy#261

Merged
stefangutica merged 2 commits into
mainfrom
remove-promisfy
May 20, 2026
Merged

remove promisfy#261
stefangutica merged 2 commits into
mainfrom
remove-promisfy

Conversation

@stefangutica
Copy link
Copy Markdown
Contributor

Reasoning

  • Eliminate Node DEP0174 warning
  • Modernize multi.exec usage
  • Keep metrics timing reliable

Proposed Changes

  • Replace promisify(multi.exec) with await multi.exec()
  • Record MULTI metric in finally
  • Small, localized change in redis-cache.service.ts

How to test

  • Run cache tests: cd packages/cache && npm test
  • Trigger bulk cache ops (e.g. setMany, deleteByPattern)
  • Verify no DEP0174 warning and MULTI metric logged

@stefangutica stefangutica merged commit 1b652b1 into main May 20, 2026
2 checks passed
@stefangutica stefangutica deleted the remove-promisfy branch May 20, 2026 14:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants