Skip to content

docs: clarify allowance nonce comment#283

Closed
wyf027 wants to merge 1 commit into
Uniswap:mainfrom
wyf027:docs/clarify-allowance-nonce
Closed

docs: clarify allowance nonce comment#283
wyf027 wants to merge 1 commit into
Uniswap:mainfrom
wyf027:docs/clarify-allowance-nonce

Conversation

@wyf027

@wyf027 wyf027 commented May 18, 2026

Copy link
Copy Markdown

Summary

  • clarifies that the packed allowance nonce is updated by allowance-based permits
  • avoids confusing it with SignatureTransfer unordered nonces

Addresses the first item in #242.

Test Plan

  • git diff --check
  • forge fmt --check src/interfaces/IAllowanceTransfer.sol was not run locally because forge is not installed in this environment; this PR only changes a NatSpec comment.

@wyf027 wyf027 closed this by deleting the head repository Jun 23, 2026
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.

1 participant