Libsass: permintaan fitur: : Tambahkan bendera untuk mematikan PERINGATAN PENGHENTIAN

Dibuat pada 31 Jan 2019  ·  10Komentar  ·  Sumber: sass/libsass

Sejak versi 4.9 dari node-sass , peringatan saat ini ditampilkan di banyak proyek. ( Anda mungkin melihat masalah terbuka yang meminta cara untuk membungkam peringatan di sini )

DEPRECATION WARNING: Passing a string to call() is deprecated and will be illegal
in Sass 4.0. Use call(get-function("variable-exists")) instead.

Peringatannya ok dan saya ingin memperbaiki semua panggilan fungsi saya yang sudah usang tetapi saya tidak bisa, alasan utamanya adalah saya bahkan tidak tahu perpustakaan mana yang bermasalah ... (bukan kode saya) saya memahami kebutuhan untuk memaksa semua orang memperbarui barang-barang mereka yang sudah usang, tetapi itu tidak layak. Saya pikir setiap pengembang bertanggung jawab atas aplikasi mereka sendiri dan mengambil risiko menjadi usang adalah tanggung jawab mereka sendiri, saya menerima tanggung jawab :)

Jadi... permintaan fitur ini adalah untuk menambahkan suatu tempat, entah bagaimana, sebuah bendera yang dapat dihidupkan/dimatikan DEPRECATION WARNINGS

Saya sudah salah mencoba menambahkan variabel lingkungan di proyek node-sass, tetapi @nschonni memberi tahu saya bahwa PR harus dilakukan di sini di libsass , namun @xzyfer memberi tahu saya bahwa solusi yang dapat diterima untuk membungkam penghentian peringatan adalah untuk memperbaiki kode usang, bahkan berpikir bahwa kode itu milik orang lain, perpustakaan.

  • Anda dapat melihat kode untuk menambahkan flag yang sudah tersedia di PR-2587 dari proyek node-sass , tidak banyak.
  • Juga masalah terbuka mengenai masalah ini adalah 2334 (juga dari proyek node-sass).

Semua 10 komentar

Saya pikir saya punya solusi yang lebih standar, apa pendapat Anda semua tentang menerapkan level log?? (seperti log4j, winston , dll).

Tolong beri tahu saya jika Anda lebih setuju tentang ini :) (jadi saya akan memodifikasi judul FR ini)

Kami tidak akan menambahkan level logging, kami juga tidak akan mengaktifkan pembungkaman peringatan penghentian. Peringatan ini sangat penting bagi kode pengguna untuk terus bekerja.

Namun kami akan mempertimbangkan solusi yang membuatnya lebih mudah untuk menemukan kode menggunakan perilaku yang tidak digunakan lagi.

Terima kasih telah menjawab semua masalah saya dengan @xzyfer ini... btw, saya telah menemukan perpustakaan dengan masalah sehingga perbaikan sedang berlangsung untuk saya :+1:

Dan jadi... saya kira semua orang menggunakan kasus itu harus menunggu sampai pemilik perpustakaan memutuskan untuk memperbaiki barang-barang usang mereka (...)

Harap diingat bahwa peringatan ini tidak terlalu penting untuk melanjutkan pekerjaan kami sebagai pengembang... jika seperti itu, kesalahan akan dilontarkan alih-alih peringatan.

Pendekatan alternatif bisa https://github.com/sass/libsass/pull/2862 ...

Ini sangat penting. Saat ini saya mendapatkan lusinan kesalahan yang sama.

DEPRECATION WARNING on line 29 of sass/elements/image.sass:
#{} interpolation near operators will be simplified in a future version of Sass.
To preserve the current behavior, use quotes:

  unquote("image_size_#{$size}")

dengan sesuatu seperti ini

+_(image_size_#{$size})

Saya bahkan tidak yakin operator apa yang dimaksud, tetapi saya lebih suka mengubah ke "versi yang disederhanakan" ketika itu datang daripada menulis alternatif yang tampak mengerikan itu.

Peringatan ini sangat penting bagi kode pengguna untuk terus bekerja.

Ya saya setuju @xzyfer , tetapi saya sudah membaca peringatan ini dan saya ingin mengesampingkannya untuk saat ini dan fokus pada hal-hal lain, tetapi saya tidak bisa karena peringatan ini memenuhi seluruh terminal

Tidak ada niat untuk mendukung ini saat ini. Peringatan penghentian sangat penting bagi kami untuk mengembangkan bahasa. Membisukan peringatan penghentian hanya menunda rasa sakit ketika perubahan yang melanggar diperkenalkan yang akan dinikmati oleh pelacak masalah kami. Untuk menghapus peringatan, turunkan versi ke versi Libsass yang mendahului pengenalan peringatan.

Peringatan hanya berguna saat pertama kali Anda membacanya. Jika seseorang mematikannya karena mereka sudah membacanya dan bosan dengannya dan karena itu menyadarinya, menunjukkannya lagi tidak akan membuat perbedaan, dan di masa depan mereka akan tahu mengapa itu rusak.
Tidak membiarkan orang menekan peringatan ini tidak membantu siapa pun, itu hanya mengganggu.

Juga peringatannya adalah tentang bagaimana itu akan menjadi lebih disederhanakan dan solusi yang diberikannya untuk membuatnya jauh lebih rumit.
Anda dapat melihat mengapa seseorang tidak ingin mengikuti saran itu, khususnya ketika saya menggunakannya di seluruh proyek saya

@xzyfer Anda punya poin bagus tentang ini !! saya setuju denganmu :)

Membisukan peringatan penghentian hanya menunda rasa sakit ketika perubahan yang melanggar diperkenalkan yang akan dinikmati oleh pelacak masalah kami

Namun saya pikir kita masih bisa menemukan solusi!! untuk sebagian besar pengembang mengeluh tentang ini. Alasan utamanya adalah bagian ini (seperti yang dikatakan @luaVolk ):

Saat ini saya mendapatkan lusinan kesalahan yang sama.

Apa pendapat Anda tentang mengurangi semua peringatan itu menjadi hanya satu ?? pada kasus saya ketika saya membuka permintaan fitur ini saya menerima sekitar 100k+ peringatan. Saya pikir Anda dapat menghapus semua "peringatan terperinci" dan cukup cetak:

Your libsass implementation is deprecated and will be illegal in Sass 4.0, for more details try '--libsass-verbose'.

Dan mungkin menambahkan:

The following errors were found:
> 12 errors found: #{} interpolation near operators will be simplified in a future version of Sass
> 4 errors found: Passing a string to call() is deprecated and will be illegal
in Sass 4.0. Use call(get-function("variable-exists")) instead.

Atau sesuatu seperti itu ... saya harap Anda mendapatkan ide, bagaimana menurut Anda @xzyfer?

Saya kira komentar terakhir saya layak untuk dikelola sebagai permintaan fitur yang berbeda (?)

Apakah halaman ini membantu?
0 / 5 - 0 peringkat