Skip to content
This repository has been archived by the owner on Jan 9, 2025. It is now read-only.

Update fee_balancer.py #1653

Closed
wants to merge 1 commit into from
Closed

Update fee_balancer.py #1653

wants to merge 1 commit into from

Conversation

Dahka2321
Copy link

Time spent on this PR: 0.2 days

Pull request type

Please check the type of change your PR introduces:

  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes
  • Other (please describe):

What is the current behavior?

The current code has several style and type safety issues:

  • Uses float division for fee calculations
  • Lacks proper type hints
  • Has formatting issues with long lines

What is the new behavior?

  • Changed float division (/) to integer division (//) for more precise fee calculations
  • Added proper type hints to get_total_balance_of_relayers function for better type safety
  • Improved code formatting by breaking long lines according to PEP 8 guidelines
  • Enhanced readability by properly structuring function parameters

These changes improve code quality and maintainability without changing the core functionality.

@Dahka2321
Copy link
Author

@ClementWalter
Fixed integer division (//)
Updated function params and type hints

@Eikix
Copy link
Member

Eikix commented Jan 8, 2025

Sorry for the late reply and lack of review. We've been pausing the development of kakarot for now.

I'm cognizant that you spent some time doing this PR, so thank you. I will have to close it since I am cleaning a bit the repo and preparing for archival.

Archiving the repo will solve issue of people working on stuff while we are not accepting new PRs.
Thanks for understanding

@Eikix Eikix closed this Jan 8, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants