Skip to content

[automatic-failover] Enforce formatting for automatic failover codebase #4176

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

ggivo
Copy link
Collaborator

@ggivo ggivo commented Jun 11, 2025

Format existing circuit breaker implementation code & enforce formatting before actual work to ensure fewer conflicts

@ggivo ggivo requested a review from atakavci June 11, 2025 06:58
@ggivo ggivo changed the title enforce formating for mcf Enforce formatting for automatic failover codebase Jun 11, 2025
@ggivo ggivo added the churn Chore: Formatting and comment cleanup (non-functional change) label Jun 11, 2025
@ggivo ggivo changed the title Enforce formatting for automatic failover codebase [automatic-failover] Enforce formatting for automatic failover codebase Jun 11, 2025
Copy link
Contributor

@atakavci atakavci left a comment

Choose a reason for hiding this comment

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

LGTM

@ggivo ggivo merged commit ee2f87f into redis:feature/automatic-failover Jun 11, 2025
14 of 30 checks passed
@ggivo ggivo deleted the automatic-failover-format branch June 16, 2025 06:37
atakavci pushed a commit to atakavci/jedis that referenced this pull request Jun 17, 2025
…se (redis#4176)

* enforce formating for mcf

* enforce formating for mcf

* Trigger integration test jobs  feature branches
atakavci pushed a commit to atakavci/jedis that referenced this pull request Jun 17, 2025
…se (redis#4176)

* enforce formating for mcf

* enforce formating for mcf

* Trigger integration test jobs  feature branches
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
churn Chore: Formatting and comment cleanup (non-functional change)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants