Auto: Versi Canary tidak ditambahkan ke beberapa PR di GitHub Action

@zephraph Pernahkah Anda melihat masalah ini muncul belakangan ini?

Aku belum. Akan tutup.

Ini bisa saja terjadi seperti ini:

1. kode dorong
2. butuh waktu lama menulis pesan pr
3. versi canary diterbitkan sebelum Anda mengirimkan PR dan mencoba mengirim komentar ke PR yang belum dibuka
4. tidak ada versi kenari yang diposting ke PR

@zephraph Ini mungkin kesempatan bagi autobot untuk menambahkan versi canary.

Saya melihat ini juga di 8.0.0 , menyalakan log menunjukkan:

✔  success   Published canary version: <details><summary>Canary Versions</summary>- `[email protected]`
- `[email protected]`</details>

Tapi komentar PR tidak diperbarui. Saya menggunakan pengaturan default auto init dengan plugin npm dan released .

Apakah PR terbuka pada saat kenari diterbitkan?

Jika Anda melakukan hal berikut, versi canary tidak akan diposting:

1. Dorong kode ke cabang baru
2. ci menjalankan build
3. terkadang setelah membangun Anda membuka PR
4. versi tidak pernah diposting kembali

Build selesai setelah saya membuat PR.

Jika itu selesai sebelum PR dibuat, ketika saya mendorong komit baru, itu masih tidak diperbarui. Apakah itu perlu bekerja pertama kali agar komit ekstra juga memperbarui deskripsi PR?

Terima kasih untuk dukungannya

Apakah itu perlu bekerja pertama kali agar komit ekstra juga memperbarui deskripsi PR?

Seharusnya tidak masalah untuk build selanjutnya. Adakah kesempatan saya bisa memeriksa repo?

Mungkin PR Anda tidak terdeteksi di platform build Anda. Saya akan menambahkan beberapa log di sini

https://github.com/intuit/auto/blob/master/packages/core/src/auto.ts#L692

Melihatnya lebih banyak, semuanya seharusnya sudah ada di log. Bisakah Anda mengaktifkan veryVerbose dan memposting log?

Kita harus dapat melihat:

1. nomor PR terdeteksi https://github.com/intuit/auto/blob/master/packages/core/src/auto.ts#L653
2. Badan PR juga akan mencatat semua fungsinya https://github.com/intuit/auto/blob/master/packages/core/src/auto.ts#L555

Jadi PR memang tidak ditemukan untuk rilis Canary (walaupun mendapat nomor PR di baris log pertama):

info      Getting commits for PR #4
302
ℹ  info      {
303
  currentBranch: 'package-2-update',
304
  isBaseBrach: false,
305
  isPR: false,
306
  shouldGraduate: true,
307
  isPrereleaseBranch: false,
308
  publishPrerelease: false
309
}
310
ℹ  info      Canary info found: { pr: undefined, build: undefined }
311
ℹ  info      Getting commits from HEAD^ to HEAD

FYI Saya menggunakan lerna dalam mode independen. Repo bersifat internal tetapi ini adalah POC, jadi saya mungkin akan memindahkannya ke publik sehingga saya dapat menunjukkan kepada Anda, akan lebih mudah untuk men-debug dengan cara ini Senin juga 😉

@hipstersmoothie Inilah repo pengujian: https://github.com/glambert/auto-lerna-independent-poc

Aku akan melihat ini besok. Terima kasih atas repo pengujiannya! Ini akan banyak membantu

Pada Minggu, 15 Desember 2019 pukul 18:32 Guillaume Lambert [email protected]
menulis:

Berikut repo pengujiannya:
https://github.com/glambert/auto-lerna-independent-poc

—
Anda menerima ini karena Anda berkomentar.
Balas email ini secara langsung, lihat di GitHub
https://github.com/intuit/auto/issues/481?email_source=notifications&email_token=AAJDEBCGLTAHIMJAI3JWYC3QY3SEHA5CNFSM4H7BEXSKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVX#HJKTDN5WWK06 ,
atau berhenti berlangganan
https://github.com/notifications/unsubscribe-auth/AAJDEBEP47ROT67TB5KOJUTQY3SEHANCNFSM4H7BEXSA
.

Ah ini mungkin terkait dengan beberapa peringatan yang saya temukan dengan tindakan GitHub.

Pustaka ini tidak mendeteksi cabang/PR dengan benar https://github.com/pvdlg/env-ci/issues/96

Getting commits for PR #4 berasal dari kami yang mencocokkan komit SHA dengan PR, sayangnya bukan info env.

Saya akan bermain-main dengannya sedikit untuk melihat apakah saya bisa membuat env-ci berperilaku seperti yang saya inginkan

Sepertinya acara push mungkin bukan yang kita inginkan?

https://mobile.twitter.com/ethomson/status/1176421835157704704

Saya memiliki PR masuk yang akan menemukan PR yang terkait dengan komit kepala untuk versi canary. Ini akan membantu menemukan PR untuk dikomentari

Ahhh benar, jadi Anda akan memiliki Push dan pull_request sebagai pemicu?

Saya akan mencoba dengan PR versi canary dan memberi tahu Anda jika berhasil 👍 pada akhirnya saya mungkin tidak menggunakan rilis canary dengan mode independen lerna karena itu akan membuat canary untuk semua paket bahkan yang tidak berubah. Tapi setidaknya saya akan tahu itu berhasil

Ahhh benar, jadi Anda akan memiliki Push dan pull_request sebagai pemicu?

Saya belum terlalu sering menggunakan tindakan GitHub jadi IDK apa alur kerja terbaiknya (semoga Anda dapat membantu kami menemukannya ). Bagi saya memiliki beberapa tindakan yang diatur agak terlalu banyak di atas kepala. Jika kenari saya berfungsi, ini sebagian besar akan meringankan masalah.

pada akhirnya saya mungkin tidak menggunakan rilis kenari dengan mode independen lerna karena itu akan membuat kenari untuk semua paket bahkan yang tidak berubah

Karena banyak pengguna sekarang telah meminta agar saya tidak memaksa menerbitkan kenari, saya terbuka untuk itu. Tetapi dalam pengalaman saya, saya telah menemukan bahwa Anda ingin memaksa menerbitkannya untuk kasus tepi.

Misalnya jika Anda mengubah konfigurasi babel dan ingin menguji perubahan pada versi canary. Tanpa penerbitan paksa lerna tidak akan mempublikasikan semua apa pun jika Anda hanya mengubah file konfigurasi global, karena tidak ada paket yang benar-benar berubah.

Apa alasan Anda tidak ingin semuanya diperbarui di kenari? Saya bukan pengguna independant jadi mungkin ada sesuatu yang saya lewatkan.

Apa alasan Anda tidak ingin semuanya diperbarui di kenari? Saya bukan pengguna independen jadi mungkin ada sesuatu yang saya lewatkan.

Kami memiliki repositori seperti 20+ lib (dan terus bertambah) sehingga melakukan rilis kenari untuk semua ini pada setiap PR akan menjadi sangat bising. Apa yang saya pikirkan adalah jika mungkin menggunakan strategi cabang next . Hal yang baik tentang menggunakan canary adalah Anda dapat menguji komit individu. Apakah ada cara untuk memicu canary "secara manual" untuk paket tertentu dalam mode independent mungkin? Misalnya: jalankan lerna changed dan buat rilis canary hanya untuk paket yang berubah. Berpikir keras di sini

tidak saat ini.

https://github.com/intuit/auto/blob/master/plugins/npm/src/index.ts#L562

Saya terbuka untuk menambahkan flag/config seperti --no-force-publish-canary . Saya ingin mempertahankan default apa adanya.

Saya terbuka untuk menambahkan flag/config seperti --no-force-publish-canary. Saya ingin mempertahankan default apa adanya

Setuju, untuk kebanyakan kasus inilah yang Anda inginkan.

:rocket: Masalah dirilis di v9.1.0 :rocket: