less/less.js

Remove PhantomJS stuff

Open

#3772 aperta il 8 gen 2023

Vedi su GitHub
 (6 commenti) (0 reazioni) (0 assegnatari)JavaScript (3517 fork)batch import
good first issueup-for-grabs

Metriche repository

Star
 (16.977 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

I saw this and though, nobody is using phantomjs. it's a headless browser that no real user is using. it was based on webkit and the development have been discontinued. think you should remove this kind of things and instead use something like puppeteer or something.

https://github.com/less/less.js/blob/eefe33a47f6fdcc228817df7435a1770ce9e51ea/packages/less/src/less-browser/index.js#L49-L56

PhantomJS is depricated and should not be used anymore.

Guida contributor

Direzione di ricerca
Esamina il file collegato in packages/less/src/less browser/index.js righe 49-56 per vedere il blocco di codice relativo a PhantomJS. Cerca nell'intera codebase eventuali altri riferimenti a PhantomJS. Il problema suggerisce di sostituire PhantomJS con Puppeteer, ma il titolo dice solo di rimuoverlo. Conferma con i manutentori se la sola rimozione è sufficiente o se è necessaria una sostituzione. Il problema è aperto da un po' di tempo, quindi controlla i commenti recenti per eventuali aggiornamenti.
Tech stack
javascriptnodejs
Dominio
build system
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScriptNode.jsless.js build system
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80