Skip to content

Conversation

@shyouhei
Copy link
Member

このPRはどういう変更を行いますか?

OpenAPI スキーマファイルを取得し、 ogen でコード生成しています

ドキュメントの変更は必要ですか?

必要ありません

Signed-off-by: 卜部 昌平 <s-urabe@sakura.ad.jp>
…encoded", "multipart/form-data"])' openapi/openapi.json | sponge openapi/openapi.json

Signed-off-by: 卜部 昌平 <s-urabe@sakura.ad.jp>
Signed-off-by: 卜部 昌平 <s-urabe@sakura.ad.jp>
…g.yml ./openapi/openapi.json

Signed-off-by: 卜部 昌平 <s-urabe@sakura.ad.jp>
Signed-off-by: 卜部 昌平 <s-urabe@sakura.ad.jp>
Signed-off-by: 卜部 昌平 <s-urabe@sakura.ad.jp>
@shyouhei shyouhei requested review from Copilot and repeatedly and removed request for repeatedly August 19, 2025 08:25
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

このPRは、OpenAPIスキーマファイルから ogen を使用してGoのAPIクライアントコードを生成する機能を追加します。

  • OpenAPIスキーマファイルの取得とogenコード生成の設定
  • 必要な依存関係の追加とその設定
  • 生成されたコードの管理とgit属性の設定

Reviewed Changes

Copilot reviewed 10 out of 17 changed files in this pull request and generated 1 comment.

File Description
openapi/README.md OpenAPIスキーマファイルの取得と変更方法の説明を追加
go.mod ogenと関連依存関係を直接的な依存として追加し、testify等も追加
apis/v1/*.go ogenによって生成されたAPIクライアントコードファイル群
.gitattributes 生成されたコードファイルをlinguist-generatedとして設定

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

suggested by copilot

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: 卜部昌平 <shyouhei@ruby-lang.org>
@shyouhei shyouhei requested review from repeatedly and yamamoto-febc and removed request for repeatedly August 19, 2025 08:30
@shyouhei shyouhei changed the title Feature/shyouhei/schema OpenAPI スキーマファイル Aug 19, 2025
@shyouhei shyouhei self-assigned this Aug 19, 2025
Base automatically changed from feature/shyouhei/foundations to main August 20, 2025 01:48
…go into feature/shyouhei/schema

Signed-off-by: 卜部 昌平 <s-urabe@sakura.ad.jp>
@shyouhei shyouhei requested review from repeatedly and removed request for yamamoto-febc August 20, 2025 02:59
Copy link

@repeatedly repeatedly left a comment

Choose a reason for hiding this comment

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

LGTM

@shyouhei shyouhei merged commit 7227a5f into main Aug 22, 2025
8 checks passed
@shyouhei shyouhei deleted the feature/shyouhei/schema branch August 22, 2025 01:25
@github-actions github-actions bot mentioned this pull request Oct 10, 2025
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