ProjectEvergreen/greenwood

consolidate all internal rollup plugin usages to Rollup config

Offen

#1.391 geöffnet am 19.01.2025

Auf GitHub ansehenContributor Guide
 (0 Kommentare) (0 Reaktionen) (1 zugewiesene Person)JavaScript (15 Forks)auto 404
CLIalpha.1enhancementgood first issuev0.35.0

Repository-Metriken

Stars
 (132 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 25T 20h) (29 gemergte PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Untersuche die beiden referenzierten Rollup Plugin Nutzungen und verschiebe sie in die Funktion getRollupConfigForBrowserScripts in rollup.config.js, wobei du sicherstellst, dass alle Importe korrekt aufgelöst werden.
Tech Stack
javascript
Domain
backendbuild system
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Rollup
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70