Haruskah pengelola github menulis ulang permintaan tarik penulis?


44

Saya bukan seorang programmer, tetapi saya melakukan beberapa pengkodean dan telah menggunakan beberapa github. Saya telah menemukan apa yang saya temukan sebagai situasi yang mengejutkan. Saya sangat akrab dengan git.

Ada proyek yang saya temukan bug (kecil) yang mempengaruhi saya. Saya menghabiskan sore mencari dan memperbaikinya. Saya bercabang repositori, melakukan perubahan, dan mengeluarkan permintaan tarik. Setelah melihat bahwa itu ditutup sebagai "Bergabung ke cabang pengembangan" Saya pikir semuanya baik-baik saja.

Saya sedang menjelajah repo hari ini bersiap-siap untuk menghapus cabang saya, dan saya tidak dapat menemukan di mana komit tersebut digabung ke dalam repo pengelola sama sekali. Setelah beberapa waktu saya menyadari itu telah ditambahkan sebagai komit, tetapi penulis bukan lagi saya.

Sejauh yang saya tahu satu-satunya cara untuk melakukan itu adalah secara khusus menggunakan rebase, amand, atau penulisan ulang riwayat lainnya untuk menghapus penulis asli.

Ini kelihatannya sangat salah bagi saya. Paling-paling itu membingungkan, paling buruk penulis repo ini mengambil kredit untuk komitmen semua orang dan kemudian sejarah kontributor asli hilang. Sekali lagi ini adalah bug kecil, saya tidak menggunakan ini untuk resume profesional saya, sepertinya tidak jujur.

Apakah ini normal? Haruskah saya mengatakan sesuatu tentang itu?

Sunting: Perasaan umum sepertinya saya harus bertanya, jadi saya akan melakukannya pagi ini.

Sesuai permintaan di bawah ini. Saya telah memeriksa dan kode saya ada dan diterapkan secara tepat ketika saya menulisnya (termasuk komentar). Saya memverifikasi bahwa pengangkat dan penulis telah berubah. Ada satu perubahan tambahan yang ditambahkan bersamaan dengan perubahan saya. Ini adalah satu baris, yang akan mempengaruhi tambalan serta kode lainnya sebelum itu. Yaitu penambahan satu baris tidak terkait dengan bug yang saya perbaiki.

Pembaruan Tampaknya jawabannya adalah bahwa penulis memiliki cabang pengembangan dan tidak ingin menggabungkan dari cabang masternya ke dalamnya. Dia menulis ulang komit saya untuk menghindari penggabungan. Saya tidak peduli dengan banyak cabang asli b / c git yang sangat kuat untuk memilih-ceri, rebase, dan menggabungkan komit sesuai kebutuhan.

Apakah ini tipikal pada github?
Haruskah saya menghubungi pengelola proyek untuk bertanya ke cabang mana untuk menerapkan tambalan?


7
+1 untuk mengajukan pertanyaan etis tentang pengkodean :) Tertarik untuk mengetahui apakah ini adalah perilaku default, sepertinya sedikit pekerjaan ekstra bagi pengelola untuk melakukan ini. Atau apakah ini akan terjadi jika pengelola memodifikasi komit Anda sedikit setelah menerima permintaan penarikan Anda?
Sunil D.

Itu pertanyaan yang bagus. Saya tidak cukup familial dengan github untuk menjawab apa yang normal . Namun, saya pikir itu mungkin untuk memilih dengan opsi -n, melakukan modifikasi, dan kemudian melakukan. Mengubah penulis secara efektif.

4
Mungkin pengelola menerapkan perubahan Anda secara manual daripada menggabungkannya. Saya sarankan bertanya kepada pengelola tentang hal itu (tanpa menuduhnya tidak jujur).
Keith Thompson

6
Hanya untuk memperjelas, itu "penulis" yang berubah, benar? Bukan "pengendara"? Saya hanya bertanya karena perbedaannya sangat penting ketika menyangkut etika kode etik.
Christopher

2
Anda tidak mengatakan seberapa besar perbaikannya (garis kode) atau bagaimana kode dilisensikan tetapi, bisa dibilang, ini juga bisa menjadi pelanggaran hak cipta Anda.
Jaydee

Jawaban:


20

Tidak, mereka seharusnya tidak, jika bisa dihindari. Ini masalah yang dalam pengalaman saya terlalu sering terjadi. Namun saya percaya ini lebih berkaitan dengan ketidaktahuan tentang bagaimana menggunakan git dengan benar daripada seseorang yang ingin mencuri kredit.

  • Jika mereka ingin memodifikasi perubahan Anda sebelum menerapkannya ke cabang utama mereka, mereka dapat dengan mudah membuat cabang untuk perubahan Anda. Mereka kemudian dapat menambahkan komit mereka sendiri setelah Anda dan kemudian menggabungkan cabang.
  • Jika permintaan tarik Anda tidak didasarkan pada versi terbaru dari cabang utama mereka, maka mereka dapat mengeluarkan a git rebase master. Jika ada konflik, mereka dapat memilih untuk memperbaiki sendiri konflik tersebut (tanpa mengubah penulis), atau memberi Anda kesempatan untuk memperbaikinya.

Saya pikir Github dapat dan harus mencari jenis pencurian kredit yang tidak disengaja ini dan mendidik para pengelola tentang praktik terbaik bila perlu.


6

Anda telah meninggalkan beberapa detail utama di sini.

  • Jika cara Anda "memperbaiki" bug itu tidak sesuai dengan keinginan pengelola, atau bahkan salah karena bug itu dikenalkan sendiri, maka pengelola mungkin harus mengedit pekerjaan Anda sebelum melakukan. Dalam hal ini dapat dimengerti untuk mengubah penulis.

  • Seperti yang disebutkan orang lain, pengarangnya sangat berbeda dari pengendara . Seperti yang Anda mungkin sudah tahu penulis adalah orang yang benar-benar membuat komit, sedangkan komiter akan menjadi orang yang menerapkannya.

Anda harus mencermati komitmen dan memperbarui pertanyaan Anda dengan temuan Anda.


1
Saya baru saja memeriksa ini. Tidak ada perubahan pada tambalan saya. Ada perubahan 1-baris sebelumnya dan tidak terkait dengan tambalan saya yang ditambahkan pada saat yang sama.
user1585512

3

Tampaknya jawabannya adalah bahwa penulis memiliki cabang pengembangan dan tidak ingin menggabungkan dari cabang masternya ke dalamnya. Dia menulis ulang komit saya untuk menghindari penggabungan.


12
Maka mereka seharusnya digunakan git cherry-pick.
svick

3

Untuk menjawab pertanyaan Anda yang diperbarui:

Sulit untuk mengatakan apa yang khas pada github, melebihi mengatakan bahwa biasanya setiap proyek berbeda dan masing-masing memiliki alur kerja pilihan mereka sendiri. Umumnya pendekatan terbaik sebelum mengirim permintaan tarikan adalah menanyakan apa alur kerjanya atau mencoba melihat apakah Anda dapat mengetahui berdasarkan permintaan tarikan tertutup sebelumnya.

Pengalaman pribadi saya adalah jika Anda tidak bertanya, umumnya mereka akan paling baik menutup permintaan tarik tanpa komentar (kasus terburuk), atau jika mereka meninggalkan komentar yang menjelaskan prosedur apa yang meminta Anda untuk memperbarui permintaan tarik Anda. Saya akan mengatakan itu tampak aneh cara pengelola dalam kasus Anda menanganinya, tetapi itu mungkin hanya jalan paling sedikit perlawanan bagi mereka. Saya ragu itu sengaja dimaksudkan untuk mencuri kredit.

Saya sarankan Anda meminta pengelola untuk menambahkan dokumentasi yang menjelaskan bagaimana mereka ingin menerima permintaan penarikan dan terhadap cabang apa untuk menghindari kebingungan dan kurangnya kredit di masa depan. Saya berharap lebih banyak proyek memberikan dokumentasi ini, karena saya pikir itu akan membuat orang lebih cenderung untuk berpartisipasi dalam proyek.

Dengan menggunakan situs kami, Anda mengakui telah membaca dan memahami Kebijakan Cookie dan Kebijakan Privasi kami.
Licensed under cc by-sa 3.0 with attribution required.