Skip to content

Conversation

@aozherelyeva
Copy link
Contributor

@aozherelyeva aozherelyeva commented Jun 5, 2025

Add @Retry annotation on the test function level.


Type of the change

  • New feature
  • Bug fix
  • Documentation fix
  • Tests enhancement

Checklist for all pull requests

  • The pull request has a description of the proposed change
  • I read the Contributing Guidelines before opening the pull request
  • The pull request uses develop as the base branch
  • Tests for the changes have been added
  • All new and existing tests passed
Additional steps for pull requests adding a new feature
  • An issue describing the proposed change exists
  • The pull request includes a link to the issue
  • The change was discussed and approved in the issue
  • Docs have been added / updated

@aozherelyeva aozherelyeva requested a review from skarpovdev June 5, 2025 13:40
@skarpovdev
Copy link
Contributor

Can it be re-used to replace the existing executeWithRetry and runWithRetry?
I haven't taken a closer look so it's fine if not.

@aozherelyeva
Copy link
Contributor Author

Can it be re-used to replace the existing executeWithRetry and runWithRetry? I haven't taken a closer look so it's fine if not.

I don't see why not. Let me try.

@aozherelyeva aozherelyeva requested a review from skarpovdev June 6, 2025 10:28
Copy link
Contributor

@skarpovdev skarpovdev left a comment

Choose a reason for hiding this comment

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

Perfect, thank you!

@aozherelyeva aozherelyeva merged commit ce8b814 into develop Jun 6, 2025
4 of 6 checks passed
@aozherelyeva aozherelyeva deleted the zarechneva/tests branch June 6, 2025 11:30
karloti pushed a commit to karloti/koog that referenced this pull request Jun 6, 2025
karloti pushed a commit to karloti/koog that referenced this pull request Jun 10, 2025
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