less/less.js

Remove PhantomJS stuff

Open

#3.772 aberto em 8 de jan. de 2023

Ver no GitHub
 (6 comments) (0 reactions) (0 assignees)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.

Guia do colaborador

Pilha de tecnologia
javascriptnodejs
Domain
build system
Tipo Issue
refactor
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
under 1 hour
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
stale
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
clear
Prerequisites
JavaScriptNode.jsless.js build system
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
80
Direção de pesquisa
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.