Skip to content

Conversation

@aapsi
Copy link
Contributor

@aapsi aapsi commented May 13, 2025

Description

This PR fixes a variable name inconsistency in the dequeue_withdrawal_requests function. The variable was assigned as num_dequeued but then referenced as num_dequeue in the for loop, which would cause an undefined variable reference.

Changes

  • Changed for i in range(num_dequeue): to for i in range(num_dequeued): to correctly reference the previously defined variable
  • This ensures the code correctly iterates through the intended number of withdrawal requests to be dequeued

Testing

The fix maintains the intended functionality of the withdrawal request system while preventing potential errors during execution.

@aapsi aapsi requested a review from eth-bot as a code owner May 13, 2025 13:07
@github-actions github-actions bot added c-update Modifies an existing proposal t-core labels May 13, 2025
@eth-bot
Copy link
Collaborator

eth-bot commented May 13, 2025

✅ All reviewers have approved.

@eth-bot eth-bot added the a-review Waiting on author to review label May 13, 2025
@eth-bot eth-bot enabled auto-merge (squash) May 13, 2025 20:33
Copy link
Collaborator

@eth-bot eth-bot left a comment

Choose a reason for hiding this comment

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

All Reviewers Have Approved; Performing Automatic Merge...

@eth-bot eth-bot merged commit 2f7d1a9 into ethereum:master May 13, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a-review Waiting on author to review c-update Modifies an existing proposal t-core

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants