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

CIのmisskey-js用の自動生成物の差分報告機構が正しく差分を取っていない #13065

Closed
1 task done
ikasoba opened this issue Jan 22, 2024 · 0 comments · Fixed by #13066
Closed
1 task done
Labels
⚠️bug? This might be a bug

Comments

@ikasoba
Copy link
Contributor

ikasoba commented Jan 22, 2024

💡 Summary

CIのmisskey-jsの自動生成物の差分報告機構(.github/workflows/check-misskey-js-autogen.yml)がプルリクエストから生成された自動生成物と、実際にPullRequestに添付されている自動生成物の差分を確認するべきであるが

実装上のミスでベースブランチから生成された自動生成物とプルリクエストから生成された自動生成物の差分を確認していたために正しく差分の報告ができていなかったバグがありました。

🥰 Expected Behavior

プルリクエストのheadから生成された自動生成物と、実際にPullRequestに添付されている自動生成物の差分を取る

🤬 Actual Behavior

実装上のミスでベースブランチのapi.jsonから生成された自動生成物とプルリクエストのheadから生成された自動生成物の差分を確認していたために正しく差分の報告ができていなかったバグがありました。

📝 Steps to Reproduce

  • フォークする
  • バックエンドから吐き出されるopenapiスキーマが変更されるよう改変
  • 自動生成物を生成してコミット
  • プルリクエストとして送信
  • CIが正しく動いていないことを確認できる

💻 Frontend Environment

* Model and OS of the device(s):
* Browser:
* Server URL:
* Misskey:

🛰 Backend Environment (for server admin)

* Installation Method or Hosting Service:
* Misskey:
* Node:
* PostgreSQL:
* Redis:
* OS and Architecture:

Do you want to address this bug yourself?

  • Yes, I will patch the bug myself and send a pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚠️bug? This might be a bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant