Skip to content

🐛 Fix: Retry Decorator Edge Case Raises No Exception (Hacktoberfest) #110

@Boburmirzo

Description

@Boburmirzo

🧩 Problem

In memori/utils/helpers.py, the retry decorator can raise a generic MemoriError("Retry attempts exhausted") even if there was a last exception. This could mask the real error.

📋 Steps to Fix

  • Ensure that the last exception is always raised if all attempts failed, and only raise MemoriError if no exception occurred.
  • Add a test for this edge case.

✅ Acceptance Criteria

  • Retry decorator always raises the last exception if present.
  • Edge cases properly tested.
  • Documentation updated if needed.

Labels: hacktoberfest, bug, help wanted

Refer to memori/utils/helpers.py

🤝 Hacktoberfest Details

  • This issue is part of Hacktoberfest 2025 — valid pull requests will be merged or labeled hacktoberfest-accepted.
  • Please review the CONTRIBUTING.md for contribution guidelines.
  • Follow our Code of Conduct to maintain a positive and inclusive environment.

⭐ Don’t forget to star the repo on GitHub. It really helps our community grow

Metadata

Metadata

Labels

Type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions