Skip to content

docs: fix incorrect parameter count in token-swap README (#494)#591

Open
plutohan wants to merge 1 commit into
solana-foundation:mainfrom
plutohan:fix/494-readme-param-count
Open

docs: fix incorrect parameter count in token-swap README (#494)#591
plutohan wants to merge 1 commit into
solana-foundation:mainfrom
plutohan:fix/494-readme-param-count

Conversation

@plutohan

@plutohan plutohan commented Jun 6, 2026

Copy link
Copy Markdown

Fixes #494.

The tokens/token-swap README says swap_exact_tokens_for_tokens "takes five parameters", but the function takes four — ctx, swap_a, input_amount, min_output_amount (see anchor/programs/token-swap/src/lib.rs and src/instructions/swap_exact_tokens_for_tokens.rs). The numbered list directly below the sentence already enumerates only those four, so this corrects "five" → "four".

Markdown-only change; no code or tests touched.

@dev-jodee

Copy link
Copy Markdown
Collaborator

@plutohan can you rebase, CI should be fixed now, and also we need signed commits for merging in our repos

@dev-jodee dev-jodee left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

signed commit + rebase

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.

Wrong no of parameter is tell in Tokens/token-swap/Readme

2 participants