Skip to content

program: Update instruction comments#78

Merged
febo merged 2 commits into
mainfrom
febo/tidying-comments
Jun 3, 2026
Merged

program: Update instruction comments#78
febo merged 2 commits into
mainfrom
febo/tidying-comments

Conversation

@febo
Copy link
Copy Markdown
Contributor

@febo febo commented Jun 3, 2026

Problem

There are a couple of issues in the comments:

  • Incorrect account indices in Trim and Close.
  • Extend and Trim do not specify that metadata accounts need to be mutable.

Solution

Update the comments fixing account indices and explicitly mentioning that immutable metadata accounts cannot be extended nor trimmed.

@febo febo requested review from joncinque and lorisleiva June 3, 2026 10:46
lorisleiva
lorisleiva previously approved these changes Jun 3, 2026
Copy link
Copy Markdown
Member

@lorisleiva lorisleiva left a comment

Choose a reason for hiding this comment

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

👌

joncinque
joncinque previously approved these changes Jun 3, 2026
@febo febo dismissed stale reviews from joncinque and lorisleiva via 44cd826 June 3, 2026 11:22
@febo
Copy link
Copy Markdown
Contributor Author

febo commented Jun 3, 2026

@lorisleiva @joncinque Sorry, just realised that Extend needed an update as well.

@febo febo requested review from joncinque and lorisleiva June 3, 2026 11:23
@febo febo merged commit 213e23f into main Jun 3, 2026
21 checks passed
@febo febo deleted the febo/tidying-comments branch June 3, 2026 19:45
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.

3 participants