Skip to content
This repository has been archived by the owner on May 25, 2024. It is now read-only.

migrate to typescript #226

Merged
merged 8 commits into from
Apr 15, 2024
Merged

Conversation

ubugeeei
Copy link
Member

@ubugeeei ubugeeei commented Mar 24, 2024

Todo

  • TypeScript に移行する方向で問題ないかメンバーに同意を取る (@ubugeeei)

レビュー観点

  • nr dev:hoge で全てのチャプターが壊れていない
    消すのでテキトーでいいかも
  • ハンズオンの内容が不自然でない

@ubugeeei ubugeeei marked this pull request as draft March 24, 2024 14:59
@ubugeeei ubugeeei self-assigned this Mar 24, 2024
@nalpan
Copy link
Collaborator

nalpan commented Apr 9, 2024

@ubugeeei
手元で確認したところ大丈夫そうでした、ありがとうございます!
ps. examples以下の各章のコードはメンテコストだけ考えると消したい気持ちがあります、、、

@ubugeeei ubugeeei requested review from k-miyake and shiyuu33 and removed request for k-miyake April 14, 2024 11:09
@ubugeeei ubugeeei marked this pull request as ready for review April 14, 2024 11:09
@ubugeeei ubugeeei changed the base branch from main to vuefes-2024 April 14, 2024 11:17
@ubugeeei ubugeeei mentioned this pull request Apr 14, 2024
Copy link
Collaborator

@k-miyake k-miyake left a comment

Choose a reason for hiding this comment

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

変更点はざっと確認して、サンプルはローカルで動作確認して問題なかったので Approve しますね(マージしちゃって良いかも)。

/examples は削除しちゃって良いと思います。

@ubugeeei
Copy link
Member Author

ubugeeei commented Apr 15, 2024

@k-miyake

ありがとうございます!

/examples は削除しちゃって良いと思います。

は docs/ で Snippet として参照してるところがちょいちょいあって、若干やること増えそうだったので別 issue にしてしまいました!

#228

@ubugeeei ubugeeei merged commit 7ca0057 into vuejs-jp:vuefes-2024 Apr 15, 2024
3 checks passed
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