Skip to content

feat: add rust support#332

Closed
darrenyong wants to merge 6 commits into
mainfrom
darren/add-rust-support
Closed

feat: add rust support#332
darrenyong wants to merge 6 commits into
mainfrom
darren/add-rust-support

Conversation

@darrenyong
Copy link
Copy Markdown

🚥 Resolves ISSUE_ID

🧰 Changes

The changes here adds in support for generating Rust snippets. I basically pulled all the Rust targets and client from here. Would love some input on if this is the correct way to do it or not.

🧬 QA & Testing

Provide as much information as you can on how to test what you've done.

@darrenyong darrenyong requested review from a team, domharrington, erunion and flinehan June 4, 2026 23:18
Comment thread src/targets/rust/helpers.ts Fixed
@darrenyong darrenyong marked this pull request as draft June 4, 2026 23:49
@darrenyong darrenyong marked this pull request as ready for review June 5, 2026 18:05
@darrenyong darrenyong requested a review from erunion as a code owner June 5, 2026 18:05
Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
Copy link
Copy Markdown
Member

@erunion erunion left a comment

Choose a reason for hiding this comment

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

I think this looks good! A couple requests though:

  • Can you amend one of your commits to add the original PR author of this as a co-author so they get contribution credits?
  • Can you add a Docker integration suite? multipart/form-data handling is always a problem when adding support for new libraries and languages and I just want to make sure that the snippets we're generating for it will work.

@erunion
Copy link
Copy Markdown
Member

erunion commented Jun 5, 2026

handled this in #334

@erunion erunion closed this Jun 5, 2026
@erunion erunion deleted the darren/add-rust-support branch June 5, 2026 23:34
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