less/less.js

Remove PhantomJS stuff

Open

#3 772 ouverte le 8 janv. 2023

Voir sur GitHub
 (6 commentaires) (0 réactions) (0 assignés)JavaScript (16 977 stars) (3 517 forks)batch import
good first issueup-for-grabs

Description

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.

Guide contributeur

Stack technique
javascriptnodejs
Domaine
build system
Type d'issue
refactor
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
under 1 hour
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
JavaScriptNode.jsless.js build system
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80
Direction de recherche
Examine the linked file at packages/less/src/less browser/index.js lines 49-56 to see the PhantomJS related code block. Search the entire codebase for any other references to PhantomJS. The issue suggests replacing PhantomJS with Puppeteer, but the title only says remove. Confirm with maintainers whether removal alone is sufficient or if a replacement is needed. The issue has been open for a while, so check recent comments for any updates.