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

chore: Pull Request時にapi.jsonの差分を表示するActions workflow #12090

Merged
merged 10 commits into from
Oct 20, 2023

Conversation

tai-cha
Copy link
Contributor

@tai-cha tai-cha commented Oct 20, 2023

What

GitHub Actionsを利用してコメントにheadとbase間のapi.jsonのdiffを出力するようにした

ついでに副産物としてapi.jsonのスナップショットとfull diffが90日間GitHubからDLできるようになる

Why

Additional info (optional)

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

@tai-cha tai-cha marked this pull request as draft October 20, 2023 15:17
@tai-cha tai-cha changed the title chore: Pull Request時にapi.jsonno chore: Pull Request時にapi.jsonの差分を表示するActions workflow Oct 20, 2023
@codecov
Copy link

codecov bot commented Oct 20, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (42a3489) 79.38% compared to head (78f7a7f) 79.22%.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #12090      +/-   ##
===========================================
- Coverage    79.38%   79.22%   -0.16%     
===========================================
  Files          932      930       -2     
  Lines        99724    99554     -170     
  Branches      8135     8106      -29     
===========================================
- Hits         79161    78876     -285     
- Misses       20563    20678     +115     

see 20 files with indirect coverage changes

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

@tai-cha tai-cha marked this pull request as ready for review October 20, 2023 15:42
@tai-cha
Copy link
Contributor Author

tai-cha commented Oct 20, 2023

おそらくコード上は問題ないけどmisskeyのリポジトリ上でGitHub ActionsがPRにコメントを書き込む権限がない

@tai-cha
Copy link
Contributor Author

tai-cha commented Oct 20, 2023

リポジトリの読み取り、PullRequestの読み取りとコメントを書き込む権限が必要なので少し細かめにセットするべきか

@tai-cha tai-cha added the 🛠️Dev Development of Misskey itself label Oct 20, 2023
@tai-cha
Copy link
Contributor Author

tai-cha commented Oct 20, 2023

一応動作イメージはこう

tai-cha#1 (comment)

@syuilo syuilo merged commit 3e5c55c into misskey-dev:develop Oct 20, 2023
@syuilo
Copy link
Member

syuilo commented Oct 20, 2023

👍🏻👍🏻👍🏻👍🏻

kanarikanaru pushed a commit to kanarikanaru/misskey that referenced this pull request Nov 2, 2023
)

* chore: Pull Request時にapi.jsonのdiffを出力するworkflow

* refactor: job names

* fix: set repository to get api diff

* chore: set permission to workflow

* set sleep 30s (shorter)

* chore: set label of diff

* chore: more attempts to fetch misskey

* chore: add full diff output of api.js

* chore: save full-diff to Artifact

* chore: add message to download diff Artifact
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🛠️Dev Development of Misskey itself
Projects
None yet
Development

Successfully merging this pull request may close these issues.

masterマージ時のPRでapi.jsonのdiffを出力できないか
2 participants