Skip to content

Conversation

@oosakiken1
Copy link

fix #20
サンプルを置き換えながら検証をすすめるコマンドを追加しました。

あわせて、テスト用のjsを追加しています。

@dorimiamn
Copy link
Contributor

@oosakiken1 こちらの PR ですがコンフリクトの解消を私の方で行おうと思います 🙇

@dorimiamn
Copy link
Contributor

dorimiamn commented Jul 1, 2024

@oosakiken1 お疲れ様です。すみませんが次回出勤時にクローンされたリポジトリのコラボレーターに追加していただいてもよろしいでしょうか……?

コンフリクトの解消作業は完了したのですがクローンしたリポジトリであることを失念していまして、作業内容を oosakiken1:main に反映するためには push するための権限が必要な状況です。

お手数おかけしますがよろしくお願いいたします 🙇

(私が知らないだけでもしかしたらもっと良い方法があるかもしれません……?)

@oosakiken1
Copy link
Author

oosakiken1 commented Jul 5, 2024

@dorimiamn
遅くなりすみません。コラボレーターに招待しました。
pushできるか確認お願いします

@dorimiamn
Copy link
Contributor

dorimiamn commented Jul 6, 2024

全然大丈夫です、ありがとうございます!
問題なく push できました。

@dorimiamn
Copy link
Contributor

#8 の行番号が NaN となる場合の対処は PR #28 で行っている変更が反映された後に別の PR を出して対処しようと思いますのでこの PR では対応しません。

@dorimiamn
Copy link
Contributor

私の方ではコンフリクトの解消と合わせて、#18 #19 #25 関連の変更を適用する作業を行いました。
これらの作業による動作への影響はないと考えています。

@oosakiken1
Copy link
Author

対応ありがとうございます!

@moesuke
Copy link

moesuke commented Jul 12, 2024

まだコンフリクトしてますね😅

@dorimiamn
Copy link
Contributor

すみません、再度コンフリクト解消しました!

@moesuke moesuke requested a review from hsjoihs August 29, 2024 06:48
@hsjoihs
Copy link
Member

hsjoihs commented Aug 8, 2025

全体的にとてもよいと思います! ひとつ気になった点として、update を upd と略すのはあまり慣例的ではないように思いました。そんなに文字数が減るわけでもありませんし、フルで update と綴るようにしてもよいかも?

@oosakiken1
Copy link
Author

確認ありがとうございます。略語は修正します!

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.

サンプルファイルの置き換え機能があると嬉しい

4 participants