Skip to content

HTTP response bodyとmultipart.WriterのClose処理追加#226

Closed
massongit wants to merge 7 commits intoupdate_super_linter_3from
add_close
Closed

HTTP response bodyとmultipart.WriterのClose処理追加#226
massongit wants to merge 7 commits intoupdate_super_linter_3from
add_close

Conversation

@massongit
Copy link
Copy Markdown
Member

@massongit massongit commented Dec 26, 2025

リソースリークを防ぐため、HTTP response bodyとmultipart.WriterのClose処理を追加します。
また、複雑になっていた ParseLocationWithClient の処理を分割し、共通処理を関数に切り出します。

#224 を前提としています。

massongit and others added 3 commits December 26, 2025 19:17
- エラーシャドーイングを修正(closeErr変数を使用)
- ParseLocationWithClient関数の循環的複雑度を削減(関数分割)
- gocycloとgovetのlintエラーを解消

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
- executeAndReadResponse関数を追加して重複コードを集約
- geocodePlace関数とmakeHTTPRequest関数で共通化
- コード重複率を0.37%から0%に削減

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@massongit massongit changed the base branch from main to update_super_linter_3 December 26, 2025 10:44
@massongit massongit removed the request for review from Goryudyuma December 26, 2025 10:47
@massongit massongit changed the base branch from update_super_linter_3 to main December 26, 2025 10:47
@massongit massongit marked this pull request as draft December 26, 2025 10:54
@massongit massongit changed the base branch from main to update_super_linter_3 December 26, 2025 10:55
@massongit massongit closed this Dec 26, 2025
@github-project-automation github-project-automation bot moved this from In Progress to Done in task list Dec 26, 2025
@massongit massongit deleted the add_close branch December 26, 2025 11:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

1 participant