最終課題提出 - Githubissues

ShuheiYoshidaJP commented 3 years ago

test Podsファイルをgitignoreしないとこれだけ差分が発生してしまいます。

yasyun commented 3 years ago

Podsファイルをgitignoreしないとこれだけ差分が発生してしまいます。

ファイルの一覧を見たところ、githubの方にはPodsとpodfile.lockは含まれていなかったのですが、正しく２つのファイル弾けてませんでしたでしょうか？提出する時に僕も注意してgitignoreの方も確認しながらやったのですが、ミスしていたらすみません。

ShuheiYoshidaJP commented 3 years ago

QiitaApiApp_iOSブランチのQiitaApi_Appフォルダの中にgitignoreが確認できないのですがどのディレクトリでgitignoreを設定したのでしょうか？

yasyun commented 3 years ago

QiitaApiApp_iOSブランチのQiitaApi_Appフォルダの中にgitignoreが確認できないのですがどのディレクトリでgitignoreを設定したのでしょうか？

QiitaLog_Appのディレクトリにあるgitignoreで、Podsとpodfile.lockはディレクトリに関係なく弾くように設定してあります。この記述で正しく動作しているのかはあまり自信ありませんが。。

ShuheiYoshidaJP commented 3 years ago

.gitignoreの表記ルールと例

.gitignoreについて知っておくと便利そうなことのセクションをよく読んでみてください。

ShuheiYoshidaJP commented 3 years ago

どのような解釈をして、どのような修正を施したか説明してもらってもいいですか？

yasyun commented 3 years ago

どのような解釈をして、どのような修正を施したか説明してもらってもいいですか？

.gitignoreの有効範囲はその下の階層全部で、親の階層に対しては効果がないとあったので、それぞれのディレクトリで.gitignoreを作成する必要があるのではと思い、Podsとpodfile.lockを除外するように.gitignoreを作成しました。また、親ディレクトリであるmobile_sakaiで.gitignoreを作成すれば、いちいちそれぞれの課題ディレクトリで.gitignoreを設定しなくてmぽよくなるのではないかと考えています。

ShuheiYoshidaJP commented 3 years ago

.gitignoreの有効範囲はその下の階層全部で、親の階層に対しては効果がないとあったので、それぞれのディレクトリで.gitignoreを作成する必要があるのではと思い、Podsとpodfile.lockを除外するように.gitignoreを作成しました。また、親ディレクトリであるmobile_sakaiで.gitignoreを作成すれば、いちいちそれぞれの課題ディレクトリで.gitignoreを設定しなくてmぽよくなるのではないかと考えています。

認識は正しいです。

https://github.com/shinonome-inc/mobile_sakai/pull/15/commits/c3482d525b548c3095a745b547012e63e2e04950

確認してみてください。こちらのコミットに.gitignoreが含まれているでしょうか？

yasyun commented 3 years ago

.gitignoreの有効範囲はその下の階層全部で、親の階層に対しては効果がないとあったので、それぞれのディレクトリで.gitignoreを作成する必要があるのではと思い、Podsとpodfile.lockを除外するように.gitignoreを作成しました。また、親ディレクトリであるmobile_sakaiで.gitignoreを作成すれば、いちいちそれぞれの課題ディレクトリで.gitignoreを設定しなくてmぽよくなるのではないかと考えています。

認識は正しいです。

c3482d5

確認してみてください。こちらのコミットに.gitignoreが含まれているでしょうか？

正しくできていなかったようで含まれていませんでした、すみません。今もう一度pushし直し、確認したところ正しくgitigonoreが含まれていたので、確認の方お願いします。

ShuheiYoshidaJP commented 3 years ago

OKです。 podsを削除し、再度 pod install を行ったら、差分は発生しませんでした。

ShuheiYoshidaJP commented 3 years ago

LGTMです。コンフリクトを直せたらkaoruさんにみてもらってください。

mcz9mm commented 3 years ago

選択したセルの選択した状態が残ったままなので、記事一覧に戻った直後か、記事に遷移する直前に解除してあげてください。

https://developer.apple.com/documentation/uikit/uitableview/1614989-deselectrow

Mar-19-2021 10-16-06

参考記事 https://qiita.com/trsxxii/items/8f71ca67da4c6f4b78d2

mcz9mm commented 3 years ago

OKです！

shinonome-inc / mobile_sakai

最終課題提出 #15