ProjectEvergreen/greenwood

consolidate all internal rollup plugin usages to Rollup config

Aberta

#1.391 aberto em 19 de jan. de 2025

 (0 comentário) (0 reação) (1 responsável)JavaScript (15 forks)auto 404
CLIalpha.1enhancementgood first issuev0.35.0

Métricas do repositório

Stars
 (132 estrelas)
Métricas de merge de PR
 (Mesclagem média 25d 20h) (29 fundiu PRs em 30d)

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

Guia do colaborador