Skip to content

fix: correct two comment typos in client/internal/request/request.go#450

Merged
fredbi merged 1 commit into
masterfrom
copilot/fix-malformed-comment-and-spelling-error
May 14, 2026
Merged

fix: correct two comment typos in client/internal/request/request.go#450
fredbi merged 1 commit into
masterfrom
copilot/fix-malformed-comment-and-spelling-error

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 14, 2026

Two typos in client/internal/request/request.go comments.

  • // This i by Runtime.createHttpRequest.// This is set by Runtime.createHttpRequest.
  • // SetFormParam adds a forn param// SetFormParam adds a form param
Original prompt
Please apply the following diffs and create a pull request.
Once the PR is ready, give it a title based on the messages of the fixes being applied.

[{"message":"Incomplete or malformed comment. Should be 'This is set by Runtime.createHttpRequest.' or similar.","fixFiles":[{"filePath":"client/internal/request/request.go","diff":"diff --git a/client/internal/request/request.go b/client/internal/request/request.go\n--- a/client/internal/request/request.go\n+++ b/client/internal/request/request.go\n@@ -111,7 +111,7 @@\n \t// that buildHTTP — which runs after the writer populates the payload\n \t// — can apply payload-aware fallback rules (see streamFallbackMime).\n \t//\n-\t// This i by Runtime.createHttpRequest.\n+\t// This is set by Runtime.createHttpRequest.\n \tconsumes []string\n \ttimeout  time.Duration\n \tbuf      *bytes.Buffer\n"}]},{"message":"Corrected spelling of 'forn' to 'form'.","fixFiles":[{"filePath":"client/internal/request/request.go","diff":"diff --git a/client/internal/request/request.go b/client/internal/request/request.go\n--- a/client/internal/request/request.go\n+++ b/client/internal/request/request.go\n@@ -197,7 +197,7 @@\n \treturn result\n }\n \n-// SetFormParam adds a forn param to the request.\n+// SetFormParam adds a form param to the request.\n //\n //   - when there is only 1 value provided, it will set it.\n //   - when there are several values provided, it will add all of those (no overriding).\n"}]}]

Copilot AI changed the title [WIP] Fix incomplete comment and correct spelling in request.go fix: correct two comment typos in client/internal/request/request.go May 14, 2026
Copilot finished work on behalf of fredbi May 14, 2026 21:09
Copilot AI requested a review from fredbi May 14, 2026 21:09
@fredbi fredbi marked this pull request as ready for review May 14, 2026 21:24
@fredbi fredbi force-pushed the copilot/fix-malformed-comment-and-spelling-error branch from bee812d to 4fb8ee2 Compare May 14, 2026 21:37
@codecov
Copy link
Copy Markdown

codecov Bot commented May 14, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.38%. Comparing base (3b614c3) to head (4fb8ee2).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #450      +/-   ##
==========================================
- Coverage   83.45%   83.38%   -0.07%     
==========================================
  Files          62       62              
  Lines        4387     4387              
==========================================
- Hits         3661     3658       -3     
- Misses        567      569       +2     
- Partials      159      160       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@fredbi fredbi merged commit 09c03d0 into master May 14, 2026
22 checks passed
@fredbi fredbi deleted the copilot/fix-malformed-comment-and-spelling-error branch May 14, 2026 21:43
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