Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use generate-api-json for api.json diff GitHub comment #12408

Merged

Conversation

anatawa12
Copy link
Member

What

githubのprへのコメントで使用するapi.jsonを #12402 で追加されたgenerate-api-jsonで生成するようにしました。

Why

  • 過去に何度かmisskeyの起動時間が長いことで落ちたことが何度かあるため
  • misskeyの起動を待たなくて良いためパフォーマンスが良くなる

Additional info (optional)

get-api-diff.ymlpull_requestイベントで発火するため、pull requestのheadのget-api-diff.ymlが使われるはずなため、generate-api-jsonコマンドがなくてエラーになることはないはずです。

Checklist

  • Read the contribution guide
  • Test working in a local environment
  • (If needed) Add story of storybook
  • (If needed) Update CHANGELOG.md
  • (If possible) Add tests

@anatawa12 anatawa12 marked this pull request as ready for review November 22, 2023 08:47
@syuilo syuilo requested a review from tai-cha November 22, 2023 08:47
Copy link

codecov bot commented Nov 22, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (b15f293) 78.63% compared to head (ee6219f) 78.63%.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop   #12408   +/-   ##
========================================
  Coverage    78.63%   78.63%           
========================================
  Files          176      176           
  Lines        23334    23334           
  Branches       495      495           
========================================
  Hits         18349    18349           
  Misses        4985     4985           

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

@anatawa12 anatawa12 force-pushed the use-generate-api-json-for-github-comment branch from 3cb1d47 to c4a16bf Compare November 22, 2023 08:50
@anatawa12
Copy link
Member Author

フロントのビルド遅いから除こうと思ったけどだめですね。フルビルドに戻しました。

Copy link
Contributor

github-actions bot commented Nov 22, 2023

このPRによるapi.jsonの差分

差分はこちら

Get diff files from Workflow Page

@tai-cha
Copy link
Contributor

tai-cha commented Nov 22, 2023

ゆくゆくやろうと思ってはいたんですが動作する条件をpackages/backend以下が変更されたらに絞ってもいいかも

@anatawa12
Copy link
Member Author

たしかにそうですね。pull_request eventのpathsフィルタでそのようにしてみます

@tai-cha
Copy link
Contributor

tai-cha commented Nov 22, 2023

本当はdiffがない場合のメッセージも変えようと思っているんですがさすがに自分でやる

Copy link
Contributor

@tai-cha tai-cha left a comment

Choose a reason for hiding this comment

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

このPRで動いているみたいなのと見た感じ問題なさそうなのでマージしても大丈夫な気がします

@syuilo syuilo merged commit 44a378c into misskey-dev:develop Nov 23, 2023
@syuilo
Copy link
Member

syuilo commented Nov 23, 2023

🙏🙏🙏

@anatawa12 anatawa12 deleted the use-generate-api-json-for-github-comment branch November 24, 2023 13:35
@anatawa12 anatawa12 mentioned this pull request Dec 3, 2023
camilla-ett pushed a commit to kaseiski/misskey that referenced this pull request Jan 2, 2024
…2408)

* ci: use generate-api-json to get api.json changes

* restore copying default.yml

* refactor: get api.json with single workflow

* ci: api.jsonのdiffをbackendが変更されたときのみ取るように
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