Less.js: नोड_मॉड्यूल में आवश्यक एक्सटेंशन

को निर्मित 12 अक्तू॰ 2017 · 23टिप्पणियाँ · स्रोत: less/less.js

मैं आज एक मुद्दे को दोहराने की कोशिश कर रहा हूं और आखिरकार मेरे पास मेरी node_modules निर्देशिका में हैकिंग करके कुछ है।

मूल रूप से, बिना एक्सटेंशन वाली LESS फ़ाइल सहित, हम यह जानते हैं... लेकिन... node_modules निर्देशिका से एक्सटेंशन के बिना विफल हो जाती है।

मैंने निम्नलिखित का प्रदर्शन करते हुए एक साधारण रेपो बनाया है: https://github.com/robhuzzey/lessimporttest

@import './one/two/three/four/test'; काम करता है

जैसा करता है: @import 'thisisinnodemodules/one/two/three/four/test.less';
(ध्यान दें कि एक एक्सटेंशन के साथ node_modules से शामिल है)

हालाँकि:

@import 'thisisinnodemodules/one/two/three/four/test';
(ध्यान दें कि बिना एक्सटेंशन के node_modules से शामिल है)

काम नहीं करता है और निम्न त्रुटि उत्पन्न करता है:

$ npm run buildStyles

> [email protected] buildStyles /Users/robert.huzzey/Sites/test/lessissue
> lessc styles.less styles.css

FileError: 'thisisinnodemodules/one/two/three/four/test' wasn't found. Tried - /Users/robert.huzzey/Sites/test/lessissue/thisisinnodemodules/one/two/three/four/test.less,/Users/robert.huzzey/Sites/test/lessissue/thisisinnodemodules/one/two/three/four/test.less,/Users/robert.huzzey/Sites/test/lessissue/node_modules/thisisinnodemodules/one/two/three/four/test,thisisinnodemodules/one/two/three/four/test.less in /Users/robert.huzzey/Sites/test/lessissue/styles.less on line 5, column 1:
4 // This does NOT work
5 <strong i="24">@import</strong> 'thisisinnodemodules/one/two/three/four/test';
6

मैंने यह दिखाने के लिए कुछ शाखाएं बनाई हैं कि यह v3 साथ एक मुद्दा प्रतीत होता है लेकिन v2 ... क्या कोई पुष्टि कर सकता है कि यह अपेक्षित या अवांछित व्यवहार है (मुझे प्रतीत नहीं हो रहा है) इसके बारे में कुछ भी ऑनलाइन खोजने के लिए)।

bug high priority

स्रोत

robhuzzey

सभी 23 टिप्पणियाँ

आज भी इस मुद्दे में भाग गया।

geuis 13 अक्तू॰ 2017

मैं इस मुद्दे की भी पुष्टि करता हूं। आप इसे इस रेपो पर क्लोनिंग और npm रन देव चलाने के लिए पुन: पेश कर सकते हैं:

https://github.com/acacha/AdminLTETinkerLaravelMix

ध्यान रखें कि यह शायद एक वेबपैक समस्या है क्योंकि लेस कमांड का उपयोग करने में कोई समस्या नहीं है।

acacha 14 अक्तू॰ 2017

@acacha मैंने वही सोचा था लेकिन मैंने अन्यथा साबित कर दिया है .. यदि आप उठाए गए मुद्दे में बनाए गए रेपो को देखते हैं ... आप देख सकते हैं कि यह वेबपैक का उपयोग नहीं कर रहा है और हमारे पास अभी भी यह समस्या है :(

robhuzzey 15 अक्तू॰ 2017

मेरे पास कम स्रोत कोड में एक खुदाई थी और यह कुछ चीजों का परीक्षण शुरू करने की संभावना थी:

https://github.com/less/less.js/blob/55380d49e96a6ed561cac4d13a774830aa3c17a3/lib/less-node/file-manager.js#L79 -L87

मैं खुदाई शुरू करने या पीआर बनाने के लिए अनिच्छुक हूं जब तक कि मुझे पता नहीं है कि यह इरादा व्यवहार नहीं है ... अगर कम के रखरखावकर्ताओं में से एक पुष्टि कर सकता है ... मुझे एक फिक्स की जांच करने में खुशी हो रही है।

robhuzzey 15 अक्तू॰ 2017

@robhuzzey node_modules से आयात करना केवल अंतिम 3.0 अल्फा में जोड़ा गया था, और नहीं, यह इरादा व्यवहार नहीं है कि यह आयात में .less बिना आयात नहीं करेगा। निश्चित रूप से 3.0 अल्फा 3 में एक बग। इसे देखने के लिए धन्यवाद।

matthew-dean 15 अक्तू॰ 2017

👍1

@ मैथ्यू-डीन मैंने इसे इस काम के साथ तय कर लिया है: https://github.com/robhuzzey/less.js/tree/issue3115_extInNodePath लेकिन यह स्पष्ट नहीं है कि इसके लिए एक परीक्षण कहां जोड़ा जाए?

इस परिवर्तन के साथ सभी परीक्षण पास हो जाते हैं... मुझे समीक्षा/सुझावों के लिए पीआर जमा करने में खुशी हो रही है?

robhuzzey 16 अक्तू॰ 2017

@robhuzzey आपको .less NPM devDependency को package.json जोड़ना होगा, और फिर एक मॉड्यूल से फ़ाइल लोड करने के लिए एक परीक्षण ( /tests/less ) जोड़ना होगा (शायद एक import-module.less फ़ाइल?) परीक्षण जोड़ने के बाद, मैं यह सत्यापित करूँगा कि यह बिना किसी सुधार के टूट जाता है यह सुनिश्चित करने के लिए कि परीक्षण काम कर रहा है।

matthew-dean 16 अक्तू॰ 2017

👍1

धन्यवाद @ मैथ्यू-डीन मैंने फिक्स में डालने से पहले मौजूदा परीक्षणों को तोड़ दिया था ... जहां मैं संघर्ष करता हूं कि सभी परीक्षणों को .less नाम दिया गया था।

मुझे लगता है कि अगली चुनौती node_modules से एक फ़ाइल शामिल करने की है।

robhuzzey 16 अक्तू॰ 2017

@robhuzzey क्या आपके पास https://github.com/less/less.js/issues/3116 देखने का भी समय होगा? यदि आप नहीं करते हैं तो ठीक है।

matthew-dean 18 अक्तू॰ 2017

@ मैथ्यू-डीन ज़रूर ... मदद करने में खुशी हुई।

मैंने यहां अपनी शाखा के लिए एक परीक्षण किया: https://github.com/robhuzzey/less.js/tree/issue3115_extInNodePath लेकिन कुछ ऐसा प्रतीत होता है जो मेरी अपेक्षा के अनुरूप काम नहीं कर रहा है (मुझे लगता है कि मैं पूरी तरह से समझ नहीं पा रहा हूं कि इसके लिए एक परीक्षण कैसे शामिल किया जाए) इस काम)।

अगर आप मुझे बता सकते हैं कि मैं क्या गलत कर रहा हूं, तो मैं इसकी सराहना करता हूं :)

robhuzzey 18 अक्तू॰ 2017

क्षमा करें @ मैथ्यू-डीन मैं इस समय विदेश में काम कर रहा हूं इसलिए अब एक अलग समय क्षेत्र पर ... आपके मुद्दों को देखकर खुशी हुई अगर आप मुझे बता सकते हैं कि इस lib के लिए परीक्षण कैसे लिखना है? मैंने समय-समय पर कुछ बार कोशिश की है लेकिन ऐसा लगता है कि यह शायद कुछ स्पष्ट है जो आप मुझे आसानी से बता सकते हैं :)

robhuzzey 19 अक्तू॰ 2017

@robhuzzey मैं file: सिंटैक्स से परिचित नहीं हूं, लेकिन, अन्यथा, यह मुझे ऐसा प्रतीत होता है जैसे इसे सही तरीके से चलाना चाहिए। केवल दूसरी चीज़ जो आपको करने की ज़रूरत है वह है import-module.css को test/css रखना, जिसे तब अपेक्षित आउटपुट के साथ तुलना फ़ाइल के रूप में उपयोग किया जाता है ताकि परीक्षण सत्यापित कर सके कि यह काम कर रहा है। जब आप grunt quicktest चलाते हैं, तो आपको अपने आउटपुट CSS में मामूली चीज़ों को समायोजित करने की आवश्यकता हो सकती है जैसे न्यूलाइन या इंडेंट केवल कम आउटपुट से मेल खाने के लिए। आप अपने पहले परीक्षण के लिए एक खाली फ़ाइल भी बना सकते हैं और फिर कंसोल लॉग में अपेक्षित आउटपुट को कॉपी/पेस्ट कर सकते हैं। जब तक यह .less छोड़े गए मॉड्यूल से नियमों को आउटपुट कर रहा है, यह महत्वपूर्ण बात है। मैं वास्तव में आपके परीक्षण को मिश्रित करूंगा ताकि कुछ (या एक) .less साथ आयात किए जाएं और कुछ बिना अधिक मजबूत परीक्षण के लिए आयात किए जाएं।

matthew-dean 19 अक्तू॰ 2017

👍1

@ मैथ्यू-डीन धन्यवाद ... अंत में यह इतना आसान था .. तुलना करने के लिए बस सीएसएस की जरूरत थी!

मुझे पहले एनपीएम में file:// सिंटैक्स के बारे में पता नहीं था ... लेकिन अब मुझे एहसास हुआ कि यह सिर्फ एक परीक्षण के लिए वास्तविक एनपीएम पैकेज बनाने के बजाय एक अच्छा उपयोग-मामला है :)

मैंने अब पीआर बना लिया है: https://github.com/less/less.js/pull/3120 मैं सर्वोत्तम विवरण के बारे में निश्चित नहीं हूं... यदि आपके पास ऐसे उदाहरण हैं जिनका मैं अनुसरण कर सकता हूं आदि या एक टेम्प्लेट जो मुझे खुशी होगी इसका पालन करें।

धन्यवाद।

robhuzzey 20 अक्तू॰ 2017

इस समस्या को हल किया गया है: https://github.com/less/less.js/pull/3120

robhuzzey 22 अक्तू॰ 2017

@robhuzzey इस फिक्स के साथ एक नया अल्फा कब जारी किया जाएगा, इस पर कोई समयरेखा? जब मैं बाहरी पुस्तकालय में चर को ओवरराइड करने का प्रयास करता हूं तो यह वर्तमान में मेरे वेबपैक बिल्ड को तोड़ देता है।

stefan-schweiger 23 अक्तू॰ 2017

@ stefan-schweiger मैं इसे आज रात या कल रात रिलीज़ करने की कोशिश करने जा रहा हूँ।

matthew-dean 23 अक्तू॰ 2017

यदि कोई कम रेपो के लिए रिलीजिंग में शामिल होने में रुचि रखता है, तो हर तरह से मुझसे यहां या ट्विटर @matthewdeaners पर संपर्क करें।

matthew-dean 23 अक्तू॰ 2017

@ stefan-schweiger क्षमा करें, केवल इसे देखा। मेरे पास रिलीज करने की क्षमता नहीं है जैसे मैथ्यू डीन के पास है (यह ईमानदार होने के लिए मेरा पहला योगदान है)।

@ मैथ्यू-डीन जैसा कि आप शायद बता सकते हैं, मैं इस समय काफी व्यस्त हूं लेकिन मेरे पास जल्द ही कुछ निर्धारित "डाउन टाइम" आ रहा है, इसलिए मैं आपके पास वापस आऊंगा यदि यह ठीक है? (मैं अभी जो कर रहा हूं उसमें और कुछ नहीं जोड़ सकता)।

robhuzzey 27 अक्तू॰ 2017

https://github.com/less/less.js/issues/3123 को ठीक करने के लिए धन्यवाद @ मैथ्यू-डीन, यही कारण है कि मुझे यह त्रुटि मेरे रिलीज बिल्ड में भी मिली। तो चिंता न करें @robhuzzey ने इसे ठीक करने से पहले मैन्युअल रूप से डाउनग्रेड करके काम किया।

stefan-schweiger 27 अक्तू॰ 2017

3.0.1 रिलीज में यह समस्या अभी भी बनी हुई है।
मैंने समस्या को हल करने के लिए मैन्युअल रूप से 2.7.3 रिलीज करने के लिए डाउनग्रेड किया।

allentcm 18 फ़र॰ 2018

👍1

@allentcm इसका कोई मतलब नहीं है। कम 2.7.3 नोड_मॉड्यूल फ़ोल्डर से बिल्कुल भी आयात नहीं करता है।

matthew-dean 19 फ़र॰ 2018

@ मैथ्यू-डीन @allentcm 2.7.3 पर डाउनग्रेड करने से मेरे लिए भी काम किया

babsonmatt 27 फ़र॰ 2018

@ मैथ्यू-डीन @allentcm @babsonmatt को 2.7.3 में डाउनग्रेड करना मेरे लिए भी काम कर गया

Hellowor1d 27 फ़र॰ 2018

क्या यह पृष्ठ उपयोगी था?

0 / 5 - 0 रेटिंग्स

Less.js: नोड_मॉड्यूल में आवश्यक एक्सटेंशन

सभी 23 टिप्पणियाँ

संबंधित मुद्दों