search

zekimazan / changedistiller

5 stars 19 forks source link

Corrections between additionalFunctionality left and right files are made #82

Closed canberksakarya closed 6 years ago

canberksakarya commented 6 years ago

Fixes #6 , #24 , #25

cagdasgerede commented 6 years ago

@canberksakarya @zekimazan

Burada ne yapilmis tam anlamadim?

"Files changed" e gore bir tane metot eklenmis bir dosyaya

resources/testdata/src_changetypes/AdditionalFunctionality_Right.java testMethod2() {}

Gercekten bu kadar mi yapilan yoksa ben mi yanlis goruyorum?

zekimazan commented 6 years ago

Daha önceki 2 pull request te değişiklikler mevcut.

8 Nisan 2018 16:47 tarihinde Cagdas Evren Gerede notifications@github.com yazdı:

@canberksakarya https://github.com/canberksakarya @zekimazan https://github.com/zekimazan

Burada ne yapilmis tam anlamadim?

"Files changed" e gore bir tane metot eklenmis bir dosyaya

resources/testdata/src_changetypes/AdditionalFunctionality_Right.java testMethod2() {}

Gercekten bu kadar mi yapilan yoksa ben mi yanlis goruyorum?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/zekimazan/changedistiller/pull/82#issuecomment-379551954, or mute the thread https://github.com/notifications/unsubscribe-auth/AYIRN9BFYs0WZcdNahzknsx-WNGK4b9fks5tmhT9gaJpZM4TC8wh .

canberksakarya commented 6 years ago

Sayın Hocam, Söz konusu olan issue üzerinde sadece "add unit test for ADDITIONAL_FUNCTIONALITY" yazmakta idi. İlk başta ben de anlayamadığım için Zeki Hocamız ile irtibata geçtim. Kendisinin bana yaptığı açıklama üzerine bu issue nun çözümü olarak böyle bir gereksinim olduğu kanısına vardım ve bu şekilde yazdım. Diğer issue larım da dahil olmak üzere toplam üç adet bana verilmiş olan issue ları son teslim tarihini de geçirmemekle beraber tamamlamış olarak pull request yaptım. Zeki hocanın da yorumları ile beraber ortaya çıkan sorunları da yine son teslim tarihini geçirmeden giderdim. Bütün issue çözümlerim de merge edildiği, görevlendirilmiş olduğum üç issue da kapatılmış olduğu için ve Zeki hocamız da son pull requestim üzerine bir sıkıntı belirtmeden merge ettiği için bir sorun olmadığını varsaydım. Saygılarımla. Canberk Sakarya.

canberksakarya commented 6 years ago

Sayın Hocam, Öncelikle geçirmiş olduğumuz dönem içerisindeki yardımlarınız için çok teşekkür ederim ve saygılarımı sunarım. Çağdaş hocamızın benim pull requestim üzerine yaptığı yorumu tam anlayamamış bulunmaktayım. Yine de kendimi ve durumumu açıklayacak şekilde bir yorum da ben ilave etmiş bulunmaktayım. Bir sıkıntı olup olmadığını anlayamadığım için size bu maili yazmaktayım. Yapmam gereken bir şey var mı acaba? Benim pull requestimi siz de merge edince ben herşey tamam olarak düşünmüştüm. Endişelenmem gereken ya da yapmam gereken bir şey olursa beni bilgilendirirseniz çok memnun olurum.

Büyük saygı ve teşekkürlerimi sunarım İyi günler dilerim. Canberk Sakarya 131101078

On Sun, Apr 1, 2018, 22:58 Zeki Mazan notifications@github.com wrote:

Merged #82 https://github.com/zekimazan/changedistiller/pull/82.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/zekimazan/changedistiller/pull/82#event-1551224730, or mute the thread https://github.com/notifications/unsubscribe-auth/AOpzns-Xs7hers__SFX--mSqsMcS2RwEks5tkTFhgaJpZM4TC8wh .