Libsass: 3.5.3 regresi: mengimpor file `.css` gagal alih-alih peringatan (seperti yang ditunjukkan oleh catatan rilis)

Dibuat pada 24 Apr 2018 · 6Komentar · Sumber: sass/libsass

Awalnya dilaporkan di sini: https://github.com/sass/libsass-python/issues/245

masukan.scss

<strong i="8">@import</strong> 'cssfile';

cssfile.css

a { b: c; }

libsass 3.5.2 (melalui libsass-python )

$ pysassc --version
pysassc 0.14.2 (sass/libsass 3.5.2)
$ pysassc input.scss 
a {
  b: c; }

libsass 3.5.3 (melalui libsass-python )

$ pysassc --version
pysassc 0.14.3 (sass/libsass 3.5.3)
$ pysassc input.scss 
pysassc: error: Error: File to import not found or unreadable: cssfile.
        on line 1 of input.scss
>> <strong i="20">@import</strong> 'cssfile';

   ^

Saya mengharapkan peringatan yang dicatat dalam catatan rilis

Sumber

asottile

Komentar yang paling membantu

Saya sangat bingung dengan depresiasi, sepertinya tidak ada yang jelas:

Peringatan penghentian akan dihapus di 3.6.0.

vs.

Menyertakan file .css dengan @import adalah perilaku non-standar yang akan dihapus di versi LibSass mendatang. Gunakan importir khusus untuk mempertahankan perilaku ini. Periksa dokumentasi implementasi Anda tentang cara membuat importir khusus.

Apakah peringatan penyusutan akan dihapus di 3.6.0 dan sass_option_push_import_extension terus berfungsi atau akankah fungsi mengimpor css sepenuhnya dihapus?

Secara pribadi, saya akan mengatakan itu akan menjadi langkah mundur yang besar bagi libsass untuk berhenti mendukung impor css, ini berarti:

perpustakaan yang menggunakan libsass masing-masing harus mengimplementasikan importir khusus mereka sendiri untuk mendukung css
atau rata-rata pengguna (yang saya kira tidak tahu tentang importir khusus atau bagaimana menerapkannya) kehilangan hak untuk mengimpor css ke sass mereka, sehingga kehilangan semua manfaat dari memeriksa dan mengompresi css.

Saya tidak menggunakan ruby, saya tidak akan dan saya tidak peduli apa yang dilakukannya dan tidak. Tolong bisakah kita menyimpan sass_option_push_import_extension .

samuelcolvin pada 24 Apr 2018

❤3

Semua 6 komentar

Saya pikir ini bisa ditutup, inilah yang diharapkan oleh # 1963.

Secara pribadi, saya pikir itu seharusnya disusutkan melalui peringatan terlebih dahulu dan menyebabkan rilis kecil daripada tambalan, tetapi sudah terlambat sekarang.

samuelcolvin pada 24 Apr 2018

Maaf ini adalah perubahan yang tidak disengaja untuk rilis tambalan ini.

Preferensi saya adalah untuk maju dengan perubahan dan memperbarui catatan rilis.

Anda harus menggunakan sass_option_push_import_extension API untuk mengaktifkan kembali impor .css seperti yang saya lihat telah Anda lakukan di https://github.com/sass/libsass-python/pull/246 .

Catatan: melakukannya akan menghasilkan peringatan penghentian untuk file .css . Peringatan penghentian akan dihapus di 3.6.0.

xzyfer pada 24 Apr 2018

Saya sangat bingung dengan depresiasi, sepertinya tidak ada yang jelas:

Peringatan penghentian akan dihapus di 3.6.0.

vs.

Menyertakan file .css dengan @import adalah perilaku non-standar yang akan dihapus di versi LibSass mendatang. Gunakan importir khusus untuk mempertahankan perilaku ini. Periksa dokumentasi implementasi Anda tentang cara membuat importir khusus.

Apakah peringatan penyusutan akan dihapus di 3.6.0 dan sass_option_push_import_extension terus berfungsi atau akankah fungsi mengimpor css sepenuhnya dihapus?

Secara pribadi, saya akan mengatakan itu akan menjadi langkah mundur yang besar bagi libsass untuk berhenti mendukung impor css, ini berarti:

perpustakaan yang menggunakan libsass masing-masing harus mengimplementasikan importir khusus mereka sendiri untuk mendukung css
atau rata-rata pengguna (yang saya kira tidak tahu tentang importir khusus atau bagaimana menerapkannya) kehilangan hak untuk mengimpor css ke sass mereka, sehingga kehilangan semua manfaat dari memeriksa dan mengompresi css.

Saya tidak menggunakan ruby, saya tidak akan dan saya tidak peduli apa yang dilakukannya dan tidak. Tolong bisakah kita menyimpan sass_option_push_import_extension .

samuelcolvin pada 24 Apr 2018

❤3

Maaf karena tidak jelas. Catatan rilis sejak itu telah diperbarui agar lebih jelas.

Saya tidak menggunakan ruby, saya tidak akan menggunakannya dan saya tidak peduli apa yang dilakukannya dan tidak dilakukan

Pertama ini bukan kita vs Ruby. Sass adalah bahasa dengan spesifikasi seperti yang lain. Implementasi Ruby adalah sumber kebenaran untuk spesifikasi bahasa Sass.

Impor CSS mentah adalah fitur yang melanggar spesifikasi bahasa Sass, dan seharusnya tidak pernah ada di LibSass. Itu harus dihapus.

Karena orang-orang menjadi sangat bergantung padanya, kita perlu menghapusnya dengan cara yang bertanggung jawab.

Karena itu, kami menjadikannya sebagai perilaku keikutsertaan melalui sass_option_push_import_extension . API ini tidak akan hilang sampai ada cara yang sesuai dengan bahasa Sass untuk melakukan ini. Fitur tersebut direncanakan untuk sistem modul 4.0.

Peringatan penghentian ditambahkan di 3.5.x karena tidak ada kait yang cukup di LibSass bagi implementor untuk menghasilkan peringatan penghentian yang benar secara kinerja. Ini adalah peringatan penghentian yang akan dihapus di 3.6.

Di sinilah kami mengharapkan para pelaksana untuk menabrak jurusan mereka jika ingin mempertahankan kepatuhan spesifikasi.

xzyfer pada 24 Apr 2018

👍2

Terima kasih telah mengklarifikasi.

Saya baru saja membaca proposal modul untuk sass 4 dan kelihatannya bagus.

samuelcolvin pada 24 Apr 2018

Lihat juga https://github.com/sass/node-sass/issues/2362 untuk menangani ini di ujung node-sass.