Skip to content

Conversation

@beck042
Copy link
Contributor

@beck042 beck042 commented Jan 30, 2026

Description

Fixed a minor bug related to the use of trailing commas.
JSON requires strict adherence to syntax, so after removing the trailing
commas, the example becomes correct and fully suitable for copying
and practical use.

Issue(s) fixed

Fixes #

Preview

Checklist

  • If this PR updates or adds documentation content that changes or adds technical meaning, it has received an approval from an engineer or DevRel from the relevant team.
  • If this PR updates or adds documentation content, it has received an approval from a technical writer.

External contributor checklist

  • I've read the contribution guidelines.
  • I've created a new issue (or assigned myself to an existing issue) describing what this PR addresses.

Note

Low Risk
Low risk documentation-only change that removes a trailing comma to make the request JSON valid and copy/pasteable.

Overview
Fixes the starknet_addInvokeTransaction docs example request JSON by removing an invalid trailing comma in the resource_bounds.l2_gas field so the snippet is valid JSON.

Written by Cursor Bugbot for commit 1ebc661. This will update automatically on new commits. Configure here.

@beck042 beck042 requested review from a team as code owners January 30, 2026 07:40
@vercel
Copy link

vercel bot commented Jan 30, 2026

@beck042 is attempting to deploy a commit to the Consensys Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Contributor

@m4sterbunny m4sterbunny left a comment

Choose a reason for hiding this comment

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

LGTM

@beck042 am approving. However, pls note that, while community support is vital to healthy docs, v. minor corrections such as single comma are inefficient. Pls consider more targeted approach going forward.

@vercel
Copy link

vercel bot commented Jan 30, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
metamask-docs Ready Ready Preview, Comment Jan 30, 2026 0:02am

Request Review

@m4sterbunny m4sterbunny merged commit b82dcf0 into MetaMask:main Jan 30, 2026
17 checks passed
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.

2 participants