Auto: Validasi konfigurasi autorc dengan ketat sebelum rilis apa pun

Dibuat pada 6 Jan 2019  ·  10Komentar  ·  Sumber: intuit/auto

Apakah permintaan fitur Anda terkait dengan masalah?

Beberapa dari kami di Artsy sedang mengerjakan migrasi banyak repo untuk menggunakan rilis otomatis. Dalam sesi batch, entah bagaimana saya berhasil mengacaukan konfigurasi dan menggunakan noReleaseLabels alih-alih skipReleaseLabels pada beberapa proyek. Hal ini menyebabkan beberapa rilis yang tidak perlu.

Jelaskan solusi yang Anda inginkan

Saya pikir konfigurasi harus divalidasi secara ketat. Jika ada nilai yang bukan merupakan bagian dari konfigurasi (yaitu noReleaseLabels), proses rilis akan gagal. Konfigurasi yang tidak valid kemungkinan berarti seseorang berada dalam waktu yang buruk. Demikian pula jika objek konfigurasi tidak cocok dengan tipe yang diharapkan, dll. Sepertinya ada awal untuk skema json yang dapat digunakan untuk tujuan ini ( auto-rc.json ).

Jelaskan alternatif yang telah Anda pertimbangkan

Jika Anda ingin mengambil pendekatan yang lebih ringan, saya sarankan setidaknya memiliki perintah validate yang memeriksa validitas konfigurasi.

enhancement hacktoberfest released

Komentar yang paling membantu

Saya semua tentang kedua hal itu. Saya bisa mengerjakan ini, tetapi harus menunggu sampai akhir pekan. Aku akan santai malam ini, ha. Saya akan melihat apakah saya dapat mengatur antarmuka TypeScript ke proses json-schema. Jangan berpikir itu akan terlalu sulit. Bagian tersulit sebenarnya hanya mencari tahu apa yang harus masuk dalam skema.

Semua 10 komentar

Jest memiliki validator konfigurasi JSON - https://github.com/facebook/jest/tree/master/packages/jest-validate#jest -validate

Saya sedang mengerjakan pembuatan skema melalui quicktype .

https://app.quicktype.io?share=vgnvzBMgaixe5c9qUCD2.

Itu sebenarnya juga menghasilkan kode untuk memvalidasinya secara otomatis... https://app.quicktype.io?share=yxg2tubeMvnazWyklgkz. Kita bisa menggunakan sesuatu seperti itu?

Mungkin tidak boleh membangun sesuatu jika memerlukan manusia untuk mengakses situs web eksternal IMO, dalam Bahaya saya membuat skema JSON dari tipe TypeScript melalui modul simpul dan mengotomatiskannya pada penyebaran, maka itu tidak akan pernah keluar dari sinkronisasi

aku suka itu
Pada Tue, Jan 8, 2019 at 05:54 Orta [email protected] menulis:

Mungkin seharusnya tidak membangun sesuatu jika itu membutuhkan akses manusia dan
situs web eksternal IMO, dalam Bahaya saya hasilkan
https://github.com/danger/danger-js/blob/master/package.json#L60 JSON
skema
https://github.com/danger/danger-js/blob/master/source/danger-incoming-process-schema.json
dari tipe TypeScript melalui modul node dan mengotomatiskannya pada penyebaran, lalu
itu tidak akan pernah keluar dari sinkronisasi


Anda menerima ini karena Anda berlangganan utas ini.
Balas email ini secara langsung, lihat di GitHub
https://github.com/intuit/auto-release/issues/143#issuecomment-452306127 ,
atau matikan utasnya
https://github.com/notifications/unsubscribe-auth/ABIyBDiYG4uWFKH6IdjG862e1UPEQCr3ks5vBKMugaJpZM4ZyUU6
.

quicktype adalah paket npm, hanya memiliki antarmuka web juga. Saya setuju, itu harus otomatis.

dapatkah dibutuhkan antarmuka TS dan membuat skema JSON? itu akan luar biasa

sepertinya tautan quicktype Anda melakukan apa yang saya jelaskan. itu sangat keren! Menjadikan ini sebagai bagian dari proses rilis kami akan sangat keren.

Mungkin kita bahkan bisa membuat plugin untuk melakukan penerbitan sebagai ekstensi VSCode

Saya semua tentang kedua hal itu. Saya bisa mengerjakan ini, tetapi harus menunggu sampai akhir pekan. Aku akan santai malam ini, ha. Saya akan melihat apakah saya dapat mengatur antarmuka TypeScript ke proses json-schema. Jangan berpikir itu akan terlalu sulit. Bagian tersulit sebenarnya hanya mencari tahu apa yang harus masuk dalam skema.


:rocket: Masalah dirilis di v9.15.0 :rocket:

Apakah halaman ini membantu?
0 / 5 - 0 peringkat