API参照先を変更 #83
Merged
API参照先を変更 #83
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
具体的にはバックエンドリポジトリのmainブランチのものにした
mehm8128
reviewed
Jan 18, 2024
There was a problem hiding this comment.
流れとしてこうなっています
npm installするpostinstallというコマンド名は特別で、npm installしたときにinstallする前に実行するコマンドを指定できます。今回ここで2つのコマンドを実行していますfetch-openapiでサーバーのmainブランチからyamlファイルをfetchしてきてscriptsディレクトリ内に保存gen-apiでそれを使ってlib/apis/genratedディレクトリ内にapit定義を生成
今回、4で元にするapi定義のyamlファイルをURL指定で直接取ってくるようにしたので、3がいらなくなりそうです。なので、scripts/fetchOpenapi.jsファイルも削除しちゃってよさそうです(dashboardも同様)(それに伴ってfetch-openapiコマンドもいらなくなりそう)
|
今回の変更に伴って、ローカルのyamlファイルがなくなったことによりprismの参照先がなくなってしまっています ここもpackage.json内prismのコマンドでの参照先をapi生成と同じようにオンライン参照にしちゃう感じでいいですか？ |
|
あー |
|
CIの更新・npmコマンドの一部削除を行いましたが、正常に動作してそうです！👀 |
mehm8128
approved these changes
Jan 18, 2024
|
:haakusimasita: |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
確認よろしくお願いします！