ProjectEvergreen/greenwood

consolidate all internal rollup plugin usages to Rollup config

Ouverte

#1 391 ouverte le 19 janv. 2025

Voir sur GitHubGuide contributeur
 (0 commentaire) (0 réaction) (1 personne assignée)JavaScript (15 forks)auto 404
CLIalpha.1enhancementgood first issuev0.35.0

Métriques du dépôt

Stars
 (132 étoiles)
Métriques de merge PR
 (Merge moyen 25j 20h) (29 PRs mergées en 30 j)

Description

Type of Change

Enhancement

Summary

Currently some Greenwood plugins provide their own Rollup plugins, but I think it would make more sense if those were all just declared "top level" in rollup.config.js since we're already doing that for other Rollup plugins like CommonJS and NodeJS resolution. I often forget that Greenwood has these Rollup plugins tucked away

Details

I think both of these just can get moved / imported into getRollupConfigForBrowserScripts

  1. Node Modules - https://github.com/ProjectEvergreen/greenwood/blob/v0.31.0-alpha.5/packages/cli/src/config/rollup.config.js#L577
  2. Terser - https://github.com/ProjectEvergreen/greenwood/blob/v0.31.0-alpha.5/packages/cli/src/plugins/resource/plugin-standard-javascript.js#L76

Guide contributeur

Direction de recherche
Examinez les deux utilisations de plugins Rollup référencées et déplacez les dans la fonction getRollupConfigForBrowserScripts dans rollup.config.js, en veillant à ce que toutes les importations soient résolues correctement.
Stack technique
javascript
Domaine
backendbuild system
Type d'issue
Fonctionnalité
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.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Rollup
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70