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

upd系コマンドの追加 #22

Open
wants to merge 14 commits into
base: main
Choose a base branch
from
Open

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
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.

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