Skip to content
This repository has been archived by the owner on Jun 1, 2023. It is now read-only.

Revert "fix ogp" #5919

Merged
merged 1 commit into from
Jan 22, 2021
Merged

Revert "fix ogp" #5919

merged 1 commit into from
Jan 22, 2021

Conversation

soutaito
Copy link
Contributor

👏 解決する issue / Resolved Issues

⛏ 変更内容 / Details of Changes

  • #5918でyarn.lockを生成したところ、typeエラーでgenerateに失敗するようになったため、要因となったと思われるPRを一旦revertします。

Copy link
Collaborator

@kaizumaki kaizumaki left a comment

Choose a reason for hiding this comment

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

LGTMです

@soutaito soutaito merged commit 87c7964 into development Jan 22, 2021
@soutaito soutaito deleted the revert-5855-fix/ogp branch January 22, 2021 10:17
@mcdmaster
Copy link
Contributor

皆さんおつかれさまです。

私の出した PR #5855 のせいで、(Node.js のアップグレードに伴う)エラーが起きてしまったのであれば、私もエラーを解決すべき責任の一翼を担うとの認識です。
差し支えないタイミングで、上記に絡みこの問題が発生した原因そのもののご教示をいただけないでしょうか。

いち市民としてさも当たり前のように無償でサービスを受けるに留まらず、能動的な問題解決を図るために私はこのプロジェクトに臨んでいると思っています。
また、新しい知識を得る度に知的刺激が得られる気分もなかなか良いものです。なにしろ、vue に触れたのは本プロジェクトが最初でしたので(笑)

@kaizumaki
Copy link
Collaborator

@mcdmaster フォローありがとうございます。昨日リリース作業をおこなった際、yarn run generate:dev でtypeエラーが発生しました。同時に #5907 をマージしており、そちらのPRでyarn.lockをアップデートしていなかったのもまずかったようです(これはまったく私のミスです)。
yarn run generate:dev で起こったtypeエラーが結構複雑で、マージからのリカバリーが難しかったので #5855 をリバートすることにしました。
よければ、最新のdevelopmentを手元に持ってきてもらい、yarn installをしてみていただけますか。

@mcdmaster
Copy link
Contributor

収集した情報を元に、PR #5927 を立ててみました

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants